Skip to content

[feat][MGS][gateway] add support for sso/login path in addition to sso-login#1068

Merged
casionone merged 1 commit into
dev-2.0.0from
dev-2.0.0-sso-login
Jun 24, 2026
Merged

[feat][MGS][gateway] add support for sso/login path in addition to sso-login#1068
casionone merged 1 commit into
dev-2.0.0from
dev-2.0.0-sso-login

Conversation

@v-kkhuang

Copy link
Copy Markdown

What is the purpose of the change

Background/Problem:
Currently, the gateway module only supports the sso-login path for SSO authentication. Some systems or configurations may prefer the standard RESTful path format sso/login, but this is not currently supported, limiting integration flexibility.

Purpose of Change:
To address this problem, this PR modifies the path matching logic in the UserRestful class to support both sso-login and sso/login paths for SSO authentication.

Value/Impact:
After the change, systems can use the more standard RESTful path format sso/login for SSO authentication while maintaining backward compatibility with existing systems using sso-login.

Related issues/PRs

Related issues: close #1067
Related pr:none

Brief change log

  • Add support for sso/login path in addition to existing sso-login path
  • Update path matching logic in UserRestful to handle both path formats

Checklist

  • I have read the Contributing Guidelines on pull requests.
  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible
  • If this is a code change: I have written unit tests to fully verify the new behavior.

@v-kkhuang v-kkhuang added the enhancement New feature or request label Jun 23, 2026

@casionone casionone left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM.

@casionone casionone merged commit e3d49c0 into dev-2.0.0 Jun 24, 2026
12 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants