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.

Sign up for GitHub

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

Jump to bottom

fix(jans-auth-server): changed method HEAD to POST in validate ssa #7870

Merged
merged 2 commits into from
Feb 28, 2024
Merged

fix(jans-auth-server): changed method HEAD to POST in validate ssa #7870

merged 2 commits into from
Feb 28, 2024

Conversation

Milton-Ch
Copy link
Contributor

Prepare

Description

Target issue

  • Changed the HEAD method to POST
  • Modified endpoint path from /ssa to /ssa/validation

closes #7454

Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

@Milton-Ch
fix(jans-auth-server): changed method HEAD to POST in validate ssa
74e1847 
Signed-off-by: Milton Ch <j.milton.chambi.m@gmail.com>
@Milton-Ch Milton-Ch requested a review from moabu February 28, 2024 02:20
@mo-auto mo-auto added area-documentation Documentation needs to change as part of issue or PR comp-jans-auth-server Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Feb 28, 2024
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed for 'jans-linux-setup'

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@yuriyz yuriyz enabled auto-merge (squash) February 28, 2024 05:35
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed for 'jans-cli'

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed for 'jans-config-api-parent'

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed for 'Jans lock server parent'

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@yuriyz yuriyz merged commit 1f744cd into main Feb 28, 2024
5 checks passed
@yuriyz yuriyz deleted the fix/ssa-response-error branch February 28, 2024 08:29
yuriyz pushed a commit that referenced this pull request Nov 7, 2024
@Milton-Ch
fix(jans-auth-server): changed method HEAD to POST in validate ssa (#…
6378262 
…7870)

Signed-off-by: Milton Ch <j.milton.chambi.m@gmail.com>
Former-commit-id: 1f744cd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yuriyz yuriyz yuriyz approved these changes

@moabu moabu moabu approved these changes

@yurem yurem Awaiting requested review from yurem yurem is a code owner

Assignees
No one assigned
Labels
area-documentation Documentation needs to change as part of issue or PR comp-jans-auth-server Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix(jans-auth-server): handle ssa validation error
4 participants
@Milton-Ch @yuriyz @moabu @mo-auto