-
Notifications
You must be signed in to change notification settings - Fork 106
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
Hide Safari alert by default #608
Milestone
Comments
@ericfranz Should this error message be completely removed along with the Since Dex is enabled by default, I don't think a site would explicitly enable HTTP Basic Authentication over Dex. |
You still have to manually install Dex. But maybe we should remove it completely. |
msquee
added a commit
that referenced
this issue
Aug 5, 2020
* Dex is now the default recommendation for auth instead of Basic Auth
ericfranz
added a commit
that referenced
this issue
Aug 5, 2020
Remove Safari compatibility alert (#608)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Now that we are using Dex as the default recommendation for authentication, we should not require sites to configure OnDemand to disable the Safari alert.
The text was updated successfully, but these errors were encountered: