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

Proxy auth dialog shown on hidden window #109667

Closed
chrmarti opened this issue Oct 29, 2020 · 2 comments
Closed

Proxy auth dialog shown on hidden window #109667

chrmarti opened this issue Oct 29, 2020 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders proxy Issues regarding network proxies verified Verification succeeded
Milestone

Comments

@chrmarti
Copy link
Collaborator

Testing #109377

Using Mac native tabs (in cases this matters). I launched from the command line and 2 windows were restored with the second native tab being on top. The proxy dialog was shown on the first window.

Version: 1.51.0-insider
Commit: 2ee8b8c
Date: 2020-10-29T06:00:13.428Z
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: 8.3.110.13-electron.0
OS: Darwin x64 19.6.0

@chrmarti
Copy link
Collaborator Author

After entering the wrong credentials on the first window, the dialog showed up the second (hidden) window. 🤔

@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug proxy Issues regarding network proxies labels Oct 29, 2020
@bpasero bpasero added this to the October 2020 milestone Oct 29, 2020
@bpasero
Copy link
Member

bpasero commented Oct 29, 2020

Awesome finding! Yeah, I thought it would be clever to show the dialog in the window that does the network request but now changed that to be the focused window, same as what the old authentication code was doing.

@chrmarti chrmarti added the verified Verification succeeded label Oct 30, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Dec 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders proxy Issues regarding network proxies verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants
@bpasero @chrmarti and others