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): perform redirect_uri validation if FAPI flag is true #2500

Closed
yuriyz opened this issue Sep 29, 2022 · 0 comments · Fixed by #2502
Closed

fix(jans-auth-server): perform redirect_uri validation if FAPI flag is true #2500

yuriyz opened this issue Sep 29, 2022 · 0 comments · Fixed by #2502
Assignees
@yuriyz
Labels
comp-jans-auth-server Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Milestone
1.0.3

Comments

@yuriyz
Copy link
Contributor

yuriyz commented Sep 29, 2022 •
edited
Loading

Describe the bug

Bug was introduced in this commit
If fapi=true redirect uri validation is skipped.

Expected behavior
Redirect Uri validation must be performed correctly.

Support: 10828
@yuriyz yuriyz added kind-bug Issue or PR is a bug in existing functionality comp-jans-auth-server Component affected by issue or PR labels Sep 29, 2022
@yuriyz yuriyz added this to the 1.0.3 milestone Sep 29, 2022
@yuriyz yuriyz assigned moabu and yuriyz and unassigned moabu Sep 29, 2022
yuriyz added a commit that referenced this issue Sep 29, 2022
@yuriyz
fix(jans-auth-server): perform redirect_uri validation if FAPI flag i…
dc709fe 
…s true #2500

docs: no docs
#2500
@yuriyz yuriyz mentioned this issue Sep 29, 2022
Merged
5 tasks
yuriyz added a commit that referenced this issue Sep 29, 2022
@yuriyz
fix(jans-auth-server): perform redirect_uri validation if FAPI flag i…
aad0460 
…s true #2500 (#2502)

docs: no docs
#2500
pujavs pushed a commit that referenced this issue Sep 30, 2022
@yuriyz @pujavs
fix(jans-auth-server): perform redirect_uri validation if FAPI flag i…
e02559c 
…s true #2500 (#2502)

docs: no docs
#2500
yuriyz added a commit that referenced this issue Sep 30, 2022
@pujavs @yuriyz
@devrimyatar @nynymike
fix(config-api): rectified sortBy field for Agama resource (#2513)
bb3ac95 
* fix(config-api): removing CB and MySqlDB endpoints

* fix(config-api): removing CB and MySqlDB endpoints

* fix(orm): length check added before accessing CustomObjectAttribute values

* fix(jans-auth-server): npe in discovery if SSA endpoint is absent #2497 (#2498)

docs: no docs
#2497

* docs: add remote connection section (#2499)

* Add remote connection section

* replace domain by demoexmple.gluu.org

* fix(jans-auth-server): perform redirect_uri validation if FAPI flag is true #2500 (#2502)

docs: no docs
#2500

* Added Deployment README and Platform goals page.

* fix: jans-linux-setup upgrade to MDS3 in fido2 (#2507)

* fix(config-api): rectified sortBy field for Agama resource

* fix(config-api): rectified sortBy field for Agama resource

* fix(config-api): rectified sortBy field for Agama resource

Co-authored-by: YuriyZ <yzabrovarniy@gmail.com>
Co-authored-by: Devrim <devrimyatar@gluu.org>
Co-authored-by: Michael Schwartz <mike@gluu.org>
@mo-auto mo-auto mentioned this issue Nov 1, 2022
Merged
@mo-auto3 mo-auto3 mentioned this issue Nov 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuriyz yuriyz

Labels
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
1.0.3
2 participants
@yuriyz @moabu