Fix for viewer-private/issues/489 - HTTP Basic Auth dialog should not be present #4799
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
When visiting a site that is protected by HTTP Basic Auth, previous Viewer versions would delegate collection of username and password to the Viewer UI - both in 2D in a dialog and in 3D on a prim.
This behavior changed as of Viewer 7.2.2 after the inclusion of CEF v139 where a dialog (appears to be running in a separate process - task bar gets a new icon for each instance for example) is presented. You can still use it to enter the username and password for the site but it's (a) ugly and more importantly (b) a griefing vector - bad actors can create prims that spam URLs requiring basic auth and each will force this username/password dialog to be displayed.
This fix reverts that so that the Viewer is now responsible for presenting UI to deal with credential entry and behaves the same way it has done in the past.
Testing
Please test both in 2D (Media Floater at login page for example) and 3D (create a prim with one face set to a media texture) and visit these URLS:
user
, password:passwd
)user
, password:pass
)guest
, password:guest
)Try both entering the correct and incorrect credentials (feedback will indicate success/failure).
Note that in 2D, you have enter the credentials twice. This has always been the case.
Details captured in https://github.com/secondlife/viewer-private/issues/489