FreshRSS/NetNewsWire
1
0
Fork 0
mirror of https://github.com/Ranchero-Software/NetNewsWire.git synced 2024-12-20 14:33:04 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix bug getting Dr. Drang’s favicon.

Browse Source
This commit is contained in:
Brent Simmons 2017-11-25 13:46:48 -08:00
parent 9865bd8e51
commit 8425e9033b
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Evergreen/Favicons/FaviconURLFinder.swift
View File

@ -28,7 +28,15 @@ struct FaviconURLFinder {
return
}
let link = faviconURL(homePageURL, data)
// Use the absoluteString of the response’s URL instead of the homePageURL,
// since the homePageURL might actually have been redirected.
// Example: Dr. Drang’s feed reports the homePageURL as http://www.leancrew.com/all-this —
// but it gets redirected to http://www.leancrew.com/all-this/ — which is correct.
// This way any relative link to a favicon in the page’s metadata
// will be made absolute correctly.
let urlToUse = response.url?.absoluteString ?? homePageURL
let link = faviconURL(urlToUse, data)
callback(link)
}
}