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

BRAZIL OB FAPI Conformance tests failed for wrong error code #816

Closed
HemantKMehta opened this issue Feb 14, 2022 · 2 comments
Closed

BRAZIL OB FAPI Conformance tests failed for wrong error code #816

HemantKMehta opened this issue Feb 14, 2022 · 2 comments
Assignees
@HemantKMehta
Labels
effort-1 Relative effort required for completion of issue or PR kind-bug Issue or PR is a bug in existing functionality

Comments

@HemantKMehta
Copy link
Contributor

Following two tests in Brazil OB (JARM-PAR) tests failed for wrong error code

  1. fapi1-advanced-final-ensure-registered-redirect-uri: https://www.certification.openid.net/log-detail.html?log=HORrJo1gm1qZ6cZ&public=true
  2. fapi1-advanced-final-par-attempt-invalid-redirect_uri: https://www.certification.openid.net/log-detail.html?log=tIRKZtUnWJDnzKn&public=true

actual error code: invalid_request_redirect_uri
expected error code any one of the following.
[
"invalid_request",
"invalid_request_object"
]
@HemantKMehta HemantKMehta assigned moabu and HemantKMehta and unassigned moabu Feb 14, 2022
@HemantKMehta HemantKMehta added effort-1 Relative effort required for completion of issue or PR kind-bug Issue or PR is a bug in existing functionality labels Feb 14, 2022
@HemantKMehta
Copy link
Contributor Author

fixed it here #814

@yuriyz yuriyz mentioned this issue Feb 14, 2022
Merged
yuriyz pushed a commit that referenced this issue Feb 14, 2022
@HemantKMehta
fix(jans-auth-server): error code correction unregister redirect_uri (#…
fe4d6a0 
…814) #816

Corrected the error code for unregistered redirect URI.
@yuriyz
Copy link
Contributor

yuriyz commented Feb 14, 2022

Done in #814

@yuriyz yuriyz closed this as completed Feb 14, 2022
This was referenced Feb 14, 2022
Closed
Closed
@mo-auto mo-auto mentioned this issue Mar 1, 2022
Merged
@mo-auto3 mo-auto3 mentioned this issue Mar 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HemantKMehta HemantKMehta

Labels
effort-1 Relative effort required for completion of issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@yuriyz @moabu @HemantKMehta