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

Add User Agent Override to SAML Authentication #261

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pturmel
Copy link

@pturmel pturmel commented Oct 21, 2023

Adds "samlUserAgent" key to settings dialog. If left blank, prior behavior is unchanged.

If provided, the WebView Profile is adjusted with ::setHttpUserAgent() before opening the portal's SAML page.

Adds "samlUserAgent" key to settings dialog.  If left blank, prior
behavior is unchanged.

If provided, the WebView Profile is adjusted with ::setHttpUserAgent()
before opening the portal's SAML page.
@pturmel
Copy link
Author

pturmel commented Oct 21, 2023

Ran into a VPN that cross-checked the provided OS and OS version against the web browser user agent. (!!)

This fixed it. I did have a connection hiccup where SAML succeeded at the IDP, but gpclient reported a failure. But then auto-connected immediately upon retry. I'm not sure if this patch is missing some nuance. (I'm not an experienced Qt5 developer.)

@yuezk yuezk force-pushed the main branch 15 times, most recently from c39bf9c to 9dea81b Compare March 10, 2024 08:31
@yuezk yuezk force-pushed the main branch 4 times, most recently from e69de98 to 187ca77 Compare March 29, 2024 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant