Onebox relative icon URL bug

(Dan Fabulich) #1

This onebox has a white space in the upper left corner on Chrome, or, on Firefox, it shows a broken image icon.

The document contains a <link> tag like this:

<link href="icon.jpg?hash=daf03254ad467bb13b79bb3b5a637832" rel="icon">

That should resolve to (which exists) but Discourse is incorrectly attempting to use which doesn’t exist and gets a 404 error.

(Sam Saffron) #2

@techAPJ can you sort this out? perhaps if there is no domain just add the originating domain in the onebox lib?

(Arpit Jalan) #4

We already do that. The issue here was that the link was page relative url (icon.jpg) vs root relative url (/icon.jpg). We were not distinguishing between these two types. Now fixed via:

Thanks for bringing this in our notice @dfabulich. :+1:

(Arpit Jalan) #5