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

eIAM Integration #1422

Closed
3 of 4 tasks
sosiology opened this issue Mar 19, 2024 · 15 comments
Closed
3 of 4 tasks

eIAM Integration #1422

sosiology opened this issue Mar 19, 2024 · 15 comments
Assignees

Comments

@sosiology
Copy link
Contributor

sosiology commented Mar 19, 2024

Enhance Visualize such that users can log in using an OIDC Identity Provider – in this case a Swiss federal login. Furthermore, they should be able to save visualizations as drafts and resume these at a later point in time.

DoD:

  • eIAM login is available in Visualize on TEST
  • eIAM login is available in Visualize on INT
  • eIAM login is available in Visualize on PROD
  • A user can log in with his eIAM credentials in each of these environments and create visualisations.
@sosiology
Copy link
Contributor Author

OIDC metadata for the connection to eIAM REF environment -> Info received Rene Kreidemacher

@sosiology sosiology assigned bprusinowski and unassigned ptbrowne Apr 16, 2024
@adintegra
Copy link
Contributor

Followed up on PKCE settings with BIT today – awaiting answer.

@sosiology
Copy link
Contributor Author

Feedback BIT: Reduce request of the scope down to Open ID

@bprusinowski
Copy link
Collaborator

bprusinowski commented Apr 24, 2024

Might be solved by #1471

@sosiology
Copy link
Contributor Author

can i support with testing anything?

@bprusinowski
Copy link
Collaborator

@sosiology not yet, we need to merge #1471 first (cc @ptbrowne)

@adintegra
Copy link
Contributor

After merging #1471, looks like the Sign In currently takes the user to the Keycloak Login screen. IIRC we had this once before. Maybe a configuration issue somewhere @bprusinowski?

image

@bprusinowski
Copy link
Collaborator

@adintegra I am taking a look 👍

@bprusinowski
Copy link
Collaborator

I think that we get the correct information from Keycloak back, but have some problem with dealing with it afterwards. After a quick investigation it turns out it might be related to Prisma, and e.g. a different type of id we get from Keycloak now.

It's hard for me to debug this without having access to logs – I pushed a commit with a debug KeycloakProvider flag set to true – once it deploys on TEST, we can do several attempts to log in, and then ask Abraxas for server logs to check what's the exact problem.

@ptbrowne
Copy link
Collaborator

ptbrowne commented Apr 25, 2024

Maybe it'd be good to try and connect to abraxas servers ? It's been a long time since I have tried and it is quite painful since we have to login through a windows jump point, but it might be helpful.

@bprusinowski
Copy link
Collaborator

@ptbrowne it's a good idea, I can try to do it tomorrow 👍

@adintegra
Copy link
Contributor

Quick update: After trying out various configurations (primarily around using Keycloak as an Authentication Bridge), I believe we may not need a Keycloak instance at all, but could connect directly to the BIT OIDC trust broker. This would simplify our architecture substantially.
BIT has provided the information that their trust broker/IdP is a custom implementation of Microsoft ADFS (Active Directory FS). While there is no built-in provider for this in next-auth, there is a discussion here on how to go about implementing this. Thus, the next step would be to adjust our current implementation in line with this and test it.
/cc @bprusinowski

@bprusinowski
Copy link
Collaborator

I will try to change the implementation @adintegra and deploy to TEST to see if it works 👍 Thanks for the investigation 🙇‍♂️

@sosiology sosiology assigned adintegra and ptbrowne and unassigned bprusinowski May 29, 2024
@ptbrowne
Copy link
Collaborator

I could login in test.visualize.admin.ch with eIAM credentials, save a chart to draft then publish it 🎉 . @sosiology

@adintegra adintegra changed the title eIAM Integration on TEST eIAM Integration Jun 3, 2024
@sosiology
Copy link
Contributor Author

Closing this issue as the login is available on PROD now, please re-open if i missed something @adintegra

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants