Skip to content

[Problem/Bug]: navigationStarting bypassed by BasicAuth: Basic realm="Web access"" #5152

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

Closed
dfghj44444 opened this issue Mar 11, 2025 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@dfghj44444
Copy link

What happened?

There's an a link with basic Auth(chanllenge is Basic realm="Web access" )in this page, while I put_canceled in NavigationStartingEvent, the page navigation still goes to navigationComplete.

Importance

Important. My app's user experience is significantly compromised.

Runtime Channel

Stable release (WebView2 Runtime)

Runtime Version

134.0.3124.51

SDK Version

1.0.2210.55

Framework

Win32

Operating System

Windows 11

OS Version

23H2

Repro steps

1.Webview2 Controller add event onNavigationStart add block url: https://testpages.eviltester.com/styled/auth/basic-auth-results.html, if target url is this , ars->put_Canceled(true).

2.Compile and start test exe, start with page https://bing.com, and type "window.location.href='https://testpages.eviltester.com/styled/auth/basic-auth-results.html'" in console.

3.you'll see even the navigationStart callback blocked this navigation, the page still jump out from bing.com, and history changed

Repros in Edge Browser

No, issue does not reproduce in the corresponding Edge version

Regression

No, this never worked

Last working version (if regression)

No response

@dfghj44444 dfghj44444 added the bug Something isn't working label Mar 11, 2025
@sivMSFT
Copy link

sivMSFT commented Mar 11, 2025

@dfghj44444 if you are facing the similar issue as #5144.

If not, you can reopen it.

@sivMSFT sivMSFT closed this as completed Mar 11, 2025
@dfghj44444
Copy link
Author

dfghj44444 commented Mar 12, 2025

It's not about focus, just navigationStarting never works in this scene. @sivMSFT and I have no permission to reopen. Thus I created a new one #5155

@sivMSFT
Copy link

sivMSFT commented Mar 12, 2025

@dfghj44444, I apologize for the inconvenience caused by mistakenly closing the issue. I am reopening the issue and closing #5155

@dfghj44444
Copy link
Author

Any Update? @dhveerap

@dhveerap
Copy link
Contributor

@dfghj44444 Fix for a similar issue was already part of 137 version. Could you kindly verify in 137 Canary to check if this is resolved?

@dfghj44444
Copy link
Author

@dhveerap I tried the canary version 138, and confirmed this issue do not exist any more, this bug could be closed, thanks.

@dhveerap
Copy link
Contributor

@dfghj44444 Thanks for the confirmation. Closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants