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(core): add SAML auth request handling endpoints #6887

Open
wants to merge 1 commit into
base: yemq-add-saml-app-session-table
Choose a base branch
from

Conversation

darcyYe
Copy link
Contributor

@darcyYe darcyYe commented Dec 16, 2024

Summary

Add add SAML auth request handling endpoints:

  1. GET /api/saml/:samlAppId/authn (resolves LOG-10122)
  2. POST /api/saml/:samlAppId/authn (resolves LOG-10121)

To receive single sign on auth requests.

Testing

Tested locally. Can successfully parse SAML login request and init OIDC sign in request to Logto.

Checklist

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

@github-actions github-actions bot added the feature Cool stuff label Dec 16, 2024
Copy link

github-actions bot commented Dec 16, 2024

COMPARE TO master

Total Size Diff ⚠️ 📈 +19.12 KB

Diff by File
Name Diff
packages/core/src/constants/index.ts 📈 +84 Bytes
packages/core/src/routes/swagger/utils/operation.ts 📈 +124 Bytes
packages/core/src/saml-applications/libraries/saml-applications.ts 📈 +120 Bytes
packages/core/src/saml-applications/queries/index.ts 📈 +3.4 KB
packages/core/src/saml-applications/queries/sessions.ts 📈 +2.31 KB
packages/core/src/saml-applications/routes/anonymous.ts 📈 +6.6 KB
packages/core/src/saml-applications/routes/utils.test.ts 📈 +692 Bytes
packages/core/src/saml-applications/routes/utils.ts 📈 +2.84 KB
packages/core/src/tenants/Queries.ts 📈 +324 Bytes
packages/phrases/src/locales/en/errors/application.ts 📈 +285 Bytes
packages/phrases/src/locales/en/errors/oidc.ts 📈 +37 Bytes
packages/schemas/alterations/next-1735012422-add-saml-application-sessions-table.ts 📈 +1.64 KB
packages/schemas/src/foundations/jsonb-types/index.ts 📈 +48 Bytes
packages/schemas/src/foundations/jsonb-types/saml-application-sessions.ts 📈 +527 Bytes
packages/schemas/tables/saml_application_sessions.sql 📈 +1.5 KB

@darcyYe darcyYe force-pushed the yemq-add-saml-auth-request-handler-endpoints branch 2 times, most recently from 972d0a8 to 10a310d Compare December 16, 2024 07:13
@darcyYe darcyYe force-pushed the yemq-add-saml-auth-request-handler-endpoints branch from 10a310d to 19a6557 Compare December 24, 2024 03:50
@darcyYe darcyYe force-pushed the yemq-add-saml-app-session-table branch 2 times, most recently from 9feaea5 to e7ca100 Compare December 24, 2024 06:45
@darcyYe darcyYe force-pushed the yemq-add-saml-auth-request-handler-endpoints branch 2 times, most recently from a81a8e2 to 7193389 Compare December 24, 2024 08:43
@github-actions github-actions bot added size/xl and removed size/l labels Dec 24, 2024
@darcyYe darcyYe force-pushed the yemq-add-saml-auth-request-handler-endpoints branch from 7193389 to 157d099 Compare December 24, 2024 10:47
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