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

feat: add SAML app secret related APIs #6832

Open
wants to merge 1 commit into
base: yemq-patch-get-saml-app-by-id-apis
Choose a base branch
from

Conversation

darcyYe
Copy link
Contributor

@darcyYe darcyYe commented Nov 25, 2024

Summary

Add following APIs:

  1. DEL /saml-applications/:id/secrets/:secretId (resolves LOG-10346)
  2. PATCH /saml-applications/:id/secrets/:secretId (resolves LOG-10345)
  3. POST /saml-applications/:id/secrets (resolves LOG-10120)

Testing

Covered by CI.

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

Copy link

github-actions bot commented Nov 25, 2024

COMPARE TO master

Total Size Diff ⚠️ 📈 +18.29 KB

Diff by File
Name Diff
.changeset/early-numbers-attend.md 📈 +1.53 KB
.changeset/four-deers-divide.md 📈 +694 Bytes
.changeset/warm-pugs-divide.md 📈 +1.87 KB
.changeset/wet-nails-guess.md 📈 +855 Bytes
packages/cli/CHANGELOG.md 📈 +193 Bytes
packages/cli/package.json 0 Bytes
packages/console/CHANGELOG.md 📈 +2.77 KB
packages/console/package.json 0 Bytes
packages/core/CHANGELOG.md 📈 +4.93 KB
packages/core/package.json 📈 +182 Bytes
packages/core/src/routes/applications/application.ts 📈 +15 Bytes
packages/core/src/routes/init.ts 📈 +177 Bytes
packages/core/src/routes/swagger/utils/documents.ts 📈 +100 Bytes
packages/core/src/routes/swagger/utils/general.ts 📈 +1017 Bytes
packages/core/src/routes/well-known/well-known.openapi.ts 📈 +1 Bytes
packages/core/src/saml-applications/libraries/saml-applications.ts 📈 +3.28 KB
packages/core/src/saml-applications/libraries/utils.ts 📈 +2.34 KB
packages/core/src/saml-applications/queries/configs.ts 📈 +1.14 KB
packages/core/src/saml-applications/queries/secrets.ts 📈 +2.61 KB
packages/core/src/saml-applications/routes/index.openapi.json 📈 +8.97 KB
packages/core/src/saml-applications/routes/index.ts 📈 +6.52 KB
packages/core/src/tenants/Libraries.ts 📈 +169 Bytes
packages/core/src/tenants/Queries.ts 📈 +342 Bytes
packages/create/CHANGELOG.md 📈 +51 Bytes
packages/create/package.json 0 Bytes
packages/experience-legacy/CHANGELOG.md 📈 +544 Bytes
packages/experience-legacy/package.json 📈 +1 Bytes
packages/experience/CHANGELOG.md 📈 +544 Bytes
packages/experience/package.json 📈 +1 Bytes
packages/integration-tests/CHANGELOG.md 📈 +1.81 KB
packages/integration-tests/package.json 📈 +1 Bytes
packages/integration-tests/src/api/saml-application.ts 📈 +1.5 KB
packages/integration-tests/src/tests/api/application/application.test.ts 📈 +6 Bytes
packages/integration-tests/src/tests/api/application/saml-application.test.ts 📈 +5.4 KB
packages/phrases-experience/CHANGELOG.md 📈 +543 Bytes
packages/phrases-experience/package.json 0 Bytes
packages/phrases/CHANGELOG.md 📈 +4.55 KB
packages/phrases/package.json 0 Bytes
packages/phrases/src/locales/en/errors/application.ts 📈 +278 Bytes
packages/schemas/CHANGELOG.md 📈 +3.32 KB
packages/schemas/alterations/1.22.0-1730689363-add-account-center.ts 📈 +965 Bytes
packages/schemas/alterations/1.22.0-1731054001-init-account-center.ts 📈 +1.22 KB
packages/schemas/alterations/1.22.0-1731304920-add-support-email-and-website-to-sie-table.ts 📈 +546 Bytes
packages/schemas/alterations/1.22.0-1731377260-add-unknown-session-redirect-url-to-sie.ts 📈 +490 Bytes
packages/schemas/alterations/1.22.0-1731900596-add-saml-application-type.ts 📈 +1.88 KB
packages/schemas/alterations/1.22.0-1731900631-add-saml-app-third-party-consistency-check.ts 📈 +569 Bytes
packages/schemas/alterations/1.22.0-1731901231-add-saml-application-secrets-table.ts 📈 +1.35 KB
packages/schemas/alterations/1.22.0-1731904029-add-saml-application-configs-table.ts 📈 +1.13 KB
packages/schemas/alterations/next-1730689363-add-account-center.ts 📈 +965 Bytes
packages/schemas/alterations/next-1731054001-init-account-center.ts 📈 +1.22 KB
packages/schemas/alterations/next-1731304920-add-support-email-and-website-to-sie-table.ts 📈 +546 Bytes
packages/schemas/alterations/next-1731377260-add-unknown-session-redirect-url-to-sie.ts 📈 +490 Bytes
packages/schemas/alterations/next-1731900596-add-saml-application-type.ts 📈 +1.88 KB
packages/schemas/alterations/next-1731900631-add-saml-app-third-party-consistency-check.ts 📈 +569 Bytes
packages/schemas/alterations/next-1731901231-add-saml-application-secrets-table.ts 📈 +1.35 KB
packages/schemas/alterations/next-1731904029-add-saml-application-configs-table.ts 📈 +1.13 KB
packages/schemas/package.json 0 Bytes
packages/schemas/src/types/index.ts 📈 +39 Bytes
packages/schemas/src/types/saml-application.ts 📈 +2.25 KB
packages/schemas/tables/saml_application_configs.sql 📈 +69 Bytes
packages/toolkit/connector-kit/CHANGELOG.md 📈 +1.81 KB
packages/toolkit/connector-kit/package.json 0 Bytes
pnpm-lock.yaml 📈 +1.95 KB

@darcyYe darcyYe force-pushed the yemq-saml-applications-secrets-apis branch from 32268df to 1608501 Compare November 25, 2024 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

1 participant