fix: unique idp entity id

Author: sattvikc

src/main/java/io/supertokens/storage/postgresql/Start.java

@@ -4439,13 +4439,21 @@ public void deleteExpiredGeneratedOptions() throws StorageQueryException {
 
     @Override
     public SAMLClient createOrUpdateSAMLClient(TenantIdentifier tenantIdentifier, SAMLClient samlClient)
-            throws StorageQueryException {
+            throws StorageQueryException, io.supertokens.pluginInterface.saml.exception.DuplicateEntityIdException {
         try {
             return SAMLQueries.createOrUpdateSAMLClient(this, tenantIdentifier, samlClient.clientId, samlClient.clientSecret,
                     samlClient.ssoLoginURL, samlClient.redirectURIs.toString(), samlClient.defaultRedirectURI,
                     samlClient.idpEntityId, samlClient.idpSigningCertificate,
                     samlClient.allowIDPInitiatedLogin, samlClient.enableRequestSigning);
         } catch (SQLException e) {
+            if (e instanceof PSQLException) {
+                PostgreSQLConfig config = Config.getConfig(this);
+                ServerErrorMessage serverMessage = ((PSQLException) e).getServerErrorMessage();
+
+                if (isUniqueConstraintError(serverMessage, config.getSAMLClientsTable(), "idp_entity_id")) {
+                    throw new io.supertokens.pluginInterface.saml.exception.DuplicateEntityIdException();
+                }
+            }
             throw new StorageQueryException(e);
         }
     }

src/main/java/io/supertokens/storage/postgresql/queries/SAMLQueries.java

@@ -57,6 +57,8 @@ public static String getQueryToCreateSAMLClientsTable(Start start) {
                 + "updated_at BIGINT NOT NULL,"
                 + "CONSTRAINT " + Utils.getConstraintName(schema, tableName, null, "pkey")
                 + " PRIMARY KEY(app_id, tenant_id, client_id),"
+                + "CONSTRAINT " + Utils.getConstraintName(schema, tableName, "idp_entity_id", "key")
+                + " UNIQUE (app_id, tenant_id, idp_entity_id),"
                 + "CONSTRAINT " + Utils.getConstraintName(schema, tableName, "app_id", "fkey") + " "
                 + "FOREIGN KEY(app_id) "
                 + "REFERENCES " + Config.getConfig(start).getAppsTable() + " (app_id) ON DELETE CASCADE,"