Skip to content

New SAML 2.0 implementation for GAM #951

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

New SAML 2.0 implementation for GAM #951

wants to merge 17 commits into from

Conversation

sgrampone
Copy link
Contributor

@sgrampone sgrampone commented Mar 13, 2025 •
edited
Loading

issue:203820

This new implementation solves:

  • Diverse dependency issues that previous connector had by avoiding specific SAML libraries and using generalist libraries more commonly used and with constant maintenance. It also reduce the number of dependencies included on the dependency tree.
  • It works with JavaEE and JakartaEE by being an External Objects, it delegates the management of servlets to GeneXus Java generator.
  • Create an implementation to Net (ex Net Core). It previously didn't exists.
  • Provide a more practical way to load keys, keystores and certificates for clients who use cloud infrastructure. It does it by loading keys, keystores and certificates using Base64 encoding.
  • Load user's roles provided by the IdP on Login.

#GXSEC

@sgrampone sgrampone requested a review from iroqueta March 13, 2025 16:12
@genexusbot
Copy link
Collaborator

Cherry pick to beta failed, 1 conflicted file in commit bedd806
  • pom.xml

@genexusbot
Copy link
Collaborator

Manual cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

iroqueta
iroqueta previously approved these changes Mar 13, 2025
View reviewed changes
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta partially success

@genexusbot
Copy link
Collaborator

Cherry pick to beta failed, 1 conflicted file in commit 15b1e7d
  • pom.xml

@genexusbot
Copy link
Collaborator

Manual cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta failed, 1 conflicted file in commit 5802b39
  • pom.xml

@genexusbot
Copy link
Collaborator

Manual cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iroqueta iroqueta iroqueta left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sgrampone @genexusbot @iroqueta