-
Notifications
You must be signed in to change notification settings - Fork 60
http basic auth popup makes extension unusable #374
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
Comments
Interesting use case, we should prevent multiple popups for the same tab or host! Thanks for sharing! |
I guess that that could already solve the issue! |
@reox could you possibly share a screen recording of this? In case it helps, I use this extension for mine, https://chromewebstore.google.com/detail/screen-recorder/hniebljpgcogalllopnjokppmgbhaden Or even the url and or console log of the service worker when this happens? It would help immensely it identifying the issue, thanks. |
Here is a recording: Recording.mp4I access it from a bookmark and the lines you see are just But I think it may be a bit hard to replicate. I setup a demo site here: https://demo.reox.at username "demo", password "password". |
For what it's worth, Vivaldi prevents extensions from interacting with HTTP auth request window. |
Uhh, thanks for the info, not sure if we'll be able to do anything about that, but it's good to know... |
Interessting: Now (after a reboot and firefox update) I only get a single browserpass popup. I can close it and enter the credentials and everything works... |
Thank you @reox for the updated information. I was getting ready to look at it this weekend. If it resurfaces I'll be happy to try to duplicate and resolve it. |
So far, I did not had the problem again. I can imagine it is a combination of having the page in the browser cache but not the credentials and upgrading the extension at the same time. |
I just came here to open an issue to ask for a preference to disable httpauth dialogs. The UI is kinda modal (I think less than the actual httpauth password dialog) and gets in the way when you want to login with a password that isn't stored in (browser)pass. Now I always have to close the browserpass window first. For my part, I just want a checkbox to disable browserpass for httpauth. I could imagine that this could be handled by a more passive UI, but it's probably hard to manage without knowing if the user wants to login with browserpass, by typing the password, or by using Firefox's password manager, so a feature to just disable it when you don't need/want it would be good anyway. |
Could you please open a separate issue for this? I'd like to understand more what makes you prefer to use e.g. Firefox password manager for some portion of websites as opposed to using browserpass for everything. Thanks! |
It happened again today! But I'm still not sure how to reproduce it... Could be the case that it is caused when the website is opened via a bookmark? |
@reox is the URL of that site publicly accessible something we can use to attempt to reproduce it? |
@patgmiller The particular site I'm using is internal - however I set up the demo page with the same settings on nginx:
I added it now to my bookmarks and will try to replicate it there as well! |
Actually, I can reproduce it! I added the demo page to my bookmark list and opened it after some hours. I can see that some of the photos load but then I get several pop-ups from browserpass. It is not that severe than on my other page but I think it shows the problem. |
okay, thanks @reox for your help in reproducing the issue, I will see what I can figure out with it. |
In fact… I can reproduce it too. My issue is complicated, that's why I wasn't sure of the origin for so long.
|
I fear there is still a bug related to #370: I have a website which uses http basic auth and when I open it, I get 50+ popups from browserpass. They all hang and display nothing but the window outline. Furthermore, my CPU goes to 100% and the browserpass extension gets unusable.
I believe the issue may be that this page I want to open contains several images which all require the auth. Basically, the rule I set in nginx is:
The text was updated successfully, but these errors were encountered: