Skip to content

fix: prevent mobile to desktop version redirect#64

Merged
ArnaudLigny merged 4 commits intoArnaudLigny:masterfrom
logeen:rewrite-user-agent-header
Nov 9, 2023
Merged

fix: prevent mobile to desktop version redirect#64
ArnaudLigny merged 4 commits intoArnaudLigny:masterfrom
logeen:rewrite-user-agent-header

Conversation

@logeen
Copy link
Copy Markdown

@logeen logeen commented Nov 9, 2023

Unfortunately #62 issue has occurred again :( Facebook automatically redirects (HTTP 301 response) every https://m.facebook.com/home.php HTTP request to desktop version https://www.facebook.com/. The only solution is to rewrite User-Agent HTTP header to convince Facebook that HTTP request was sent from mobile browser.

PS: Rewriting User-Agent header simply in window.fetch function doesn't work in Chrome.

@logeen
Copy link
Copy Markdown
Author

logeen commented Nov 9, 2023

@ArnaudLigny, please take a look

@ArnaudLigny ArnaudLigny changed the title Prevent mobile to desktop version redirect fix: prevent mobile to desktop version redirect Nov 9, 2023
@ArnaudLigny ArnaudLigny self-requested a review November 9, 2023 22:36
@ArnaudLigny ArnaudLigny added the bug label Nov 9, 2023
@ArnaudLigny ArnaudLigny merged commit dbc3198 into ArnaudLigny:master Nov 9, 2023
@ArnaudLigny
Copy link
Copy Markdown
Owner

Merged and deployed, now I'm waiting for the webstore validation (because of new required permissions).

@piobaire24
Copy link
Copy Markdown

Hi @ArnaudLigny
Thanks for promptly getting onto this issue. Unfortunately the problem remains.
The F-Notifier icon just shows a question mark
image

However, clicking the icon does open the relevant Facebook page.

Thanks!
Geoff

@logeen logeen deleted the rewrite-user-agent-header branch November 11, 2023 08:22
@dan073eb
Copy link
Copy Markdown

Mine was updated a month ago and seemed to fix the issue of just seeing the question mark, but now for some reason the question mark is back. Any progress?

Thanks,
Dan

@ArnaudLigny
Copy link
Copy Markdown
Owner

Hello @dan073eb, I thinks it's related to the new policy about cookies...
Could you create a dedicated issue? Thx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants