Skip to content

Commit

Permalink
Don't try to redirect to the feed preview page
Browse files Browse the repository at this point in the history
Since 1bcf39f the feed preview page
isn't web accessible anymore, so we also can't redirect to it.
This would only break feed previews for feeds that use an actual
RSS/Atom mime-type.

Fixes #477
  • Loading branch information
evilpie committed Mar 4, 2024
1 parent e276adc commit f18de58
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions background/background.js
Original file line number Diff line number Diff line change
Expand Up @@ -453,9 +453,8 @@ chrome.webRequest.onHeadersReceived.addListener(details => {
// Atom or RSS MIME type, redirect to preview page
const type = header.value.toLowerCase().replace(/^\s+|\s*(?:;.*)?$/g, "");
if (type == "application/rss+xml" || type == "application/atom+xml") {
return {
redirectUrl: getSubscribeURL(details.url),
};
FeedPreview.show(details.tabId, details.url);
return {cancel: true};
}

// XML MIME type, try sniffing for feed content
Expand Down

0 comments on commit f18de58

Please sign in to comment.