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

chore(core): hide oidcClientMetadata of SAML apps when using GET app APIs #6979

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

darcyYe
Copy link
Contributor

@darcyYe darcyYe commented Jan 24, 2025

Summary

hide oidcClientMetadata of SAML apps when using GET app APIs (resolves LOG-10499), affect GET /applications and GET /applications/:id APIs.

Testing

Covered by integration tests.

Checklist

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

Copy link

github-actions bot commented Jan 24, 2025

COMPARE TO master

Total Size Diff 📈 +945 Bytes

Diff by File
Name Diff
packages/core/src/routes/applications/application.ts 📈 +465 Bytes
packages/integration-tests/src/tests/api/application/saml-application.test.ts 📈 +480 Bytes

@darcyYe darcyYe force-pushed the yemq-hide-oidc-client-metadata-for-saml-apps-when-using-get-apps-API branch from b684413 to c21a2ee Compare January 24, 2025 06:03
@darcyYe darcyYe changed the title chore: hide oidcClientMetadata of SAML apps when using GET app APIs chore(core): hide oidcClientMetadata of SAML apps when using GET app APIs Jan 24, 2025
@darcyYe darcyYe force-pushed the yemq-hide-oidc-client-metadata-for-saml-apps-when-using-get-apps-API branch from c21a2ee to 09f1ce2 Compare January 24, 2025 06:43
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