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

Update database setup and DAO implementation for schema changes #6014

Draft
wants to merge 2 commits into
base: feature-remove-registry-saml
Choose a base branch
from

Conversation

Osara-B
Copy link

@Osara-B Osara-B commented Oct 11, 2024

Proposed changes in this pull request

  • Add new database to H2 and update constants (IDN_SAML2_SERVICE_PROVIDER, IDN_SAML2_SP_PROPERTIES)
  • Update DAO Implementation according to updated schema

@@ -654,6 +656,14 @@ public void setIdpEntityIDAlias(String idpEntityIDAlias) {
this.idpEntityIDAlias = idpEntityIDAlias;
}

public String getSingleLogoutMethod(){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix formatting


--SAML--
CREATE INDEX IDX_SAML2_CONFIG ON IDN_SAML2_SERVICE_PROVIDER (ISSUER,TENANT_ID);
CREATE INDEX IDX_SAML2_ATTRIBUTES ON IDN_SAML2_SP_PROPERTIES (SP_ID,PROPERTY_NAME );
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a new line

@Osara-B Osara-B force-pushed the saml-feature branch 3 times, most recently from 7d2b1eb to 194f70a Compare October 14, 2024 05:50
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.

2 participants