Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix "Search LibRedirect" from context menu on Chrome #719

Merged

Conversation

GlenLowland
Copy link
Contributor

@GlenLowland GlenLowland commented May 25, 2023

When you try to select a text on a page and click a "Search LibRedirect" on Google Chrome - it does not redirect to the frontend and shows error page.
After an investigation and debug, I saw Chrome passes details.initiator = "null" for such requests.
Handle this case in code and allow redirects.

When you try to select a text on a page and click a "Search LibRedirect"
on Google Chrome - it does not redirect to the frontend and shows error page.
After an investigation and debug, I saw Chrome passes details.initiator = "null" for such requests.
Handle this case in code and allow redirects.
@GlenLowland GlenLowland changed the title Fix "Search LibRedirect" from context menu Fix "Search LibRedirect" from context menu on Chrome May 25, 2023
@ManeraKai ManeraKai merged commit b26f510 into libredirect:master May 25, 2023
@ManeraKai
Copy link
Member

Thanks! :)

@GlenLowland GlenLowland deleted the fix_search_from_context_menu_chrome branch May 25, 2023 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants