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

Single auth - double sign-in #287

Closed
eljeffeg opened this issue Nov 20, 2024 · 6 comments
Closed

Single auth - double sign-in #287

eljeffeg opened this issue Nov 20, 2024 · 6 comments
Assignees
Labels
accepted This issue was accepted, we will work on this at some point enhancement New feature or request ui-frontend

Comments

@eljeffeg
Copy link

Is your feature request related to a problem? Please describe.
We only support SAML authentication. When we hit the AssemblyLine page, we are met with this screen.

Screenshot 2024-11-20 at 11 52 47 AM

After you go through the SAML authentication, it takes you back to this unnecessary step, prompting you to "sign in".. again?

Screenshot 2024-11-20 at 11 53 30 AM

Once you click that, it signs you in.

Describe the solution you'd like
Just sign them in and forgo the second "sign-in" prompt when there is only one authentication method enabled.

@eljeffeg eljeffeg added assess We still haven't decided if this will be worked on or not enhancement New feature or request labels Nov 20, 2024
@cccs-rs
Copy link
Contributor

cccs-rs commented Nov 28, 2024

Hmm... could this be an issue if let's say you have access to multiple accounts using SAML/any OAuth service.

In a more general case, you'll likely only have one but I see the second sign-in as more of a confirmation of "Is this the user you want to sign in as or as a different user".

@eljeffeg
Copy link
Author

eljeffeg commented Nov 28, 2024

Probably a low probability, but they could always just log out and sign in with the correct account in that situation. It's just sort of odd that it confirms the account I just logged into. Just log me in. If I didn't want that account, I should have logged in with the correct account - that's not on AssemblyLine, that's on me. Canadians.. too polite. 😂

@cccs-rs cccs-rs added ui-frontend accepted This issue was accepted, we will work on this at some point and removed assess We still haven't decided if this will be worked on or not labels Nov 28, 2024
cccs-rs added a commit to CybercentreCanada/assemblyline-ui-frontend that referenced this issue Dec 5, 2024
Remove the second login prompt if there's exactly one SSO auth configured (dev)
@cccs-rs
Copy link
Contributor

cccs-rs commented Dec 5, 2024

So the patch I've made so far will still display the first prompt with "Sign in with {provider}" but if there's only one provider configured on the system, it won't display the second prompt for confirmation before login.

You can test this with the latest dev release.

cccs-rs added a commit to CybercentreCanada/assemblyline-ui-frontend that referenced this issue Dec 6, 2024
Remove the second login prompt if there's exactly one SSO auth configured
@cccs-rs
Copy link
Contributor

cccs-rs commented Dec 6, 2024

This should be patched in the 4.5.0.63 release.

@cccs-rs cccs-rs closed this as completed Dec 6, 2024
@eljeffeg
Copy link
Author

eljeffeg commented Jan 15, 2025

I'm at v 4.5.0.68, but I still get the double login. My config looks like this with just saml enabled. Do I need to delete internal or put in entries for oath and set enabled: false?

configuration:
  auth:
    internal:
      # Disable internal login, you could also leave it on if you want
      enabled: false
    saml:
      # Should SAML authentication be enabled or not
      enabled: true
      ...

@cccs-rs
Copy link
Contributor

cccs-rs commented Jan 21, 2025

Based on this commit:
CybercentreCanada/assemblyline-ui-frontend@133f9c9

It should remove the double login if you're using SAML with no other OAuth provider or if system is configured with exactly one OAuth provider. I don't think internal logins should affect the quick login.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted This issue was accepted, we will work on this at some point enhancement New feature or request ui-frontend
Projects
None yet
Development

No branches or pull requests

3 participants