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): npe in discovery if SSA endpoint is absent #2497

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

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

yuriyz opened this issue Sep 29, 2022 · 0 comments · Fixed by #2498
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

Describe the bug

Call to discovery fail due to NPE exception. Reason is that it's missed in configuration.

2022-09-29 08:41:51,286 ERROR [qtp1549725679-13] 6668bf25-e93b-455c-9773-67d770e3b582 [io.jans.as.server.servlet.OpenIdConfiguration] (OpenIdConfiguration.java:303) - null
java.lang.NullPointerException: null
	at io.jans.as.server.servlet.OpenIdConfiguration.processRequest(OpenIdConfiguration.java:288) ~[classes/:?]
	at io.jans.as.server.servlet.OpenIdConfiguration.doGet(OpenIdConfiguration.java:478) ~[classes/:?]
	at jakarta.servlet.http.HttpServlet.service(HttpServlet.java:500) ~[jetty-jakarta-servlet-api-5.0.2.jar:?]
	at jakarta.servlet.http.HttpServlet.service(HttpServlet.java:587) ~[jetty-jakarta-servlet-api-5.0.2.jar:?]
	at org.eclipse.jetty.servlet.ServletHolder$NotAsync.service(ServletHolder.java:1419) ~[?:?]
	at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:764) ~[?:?]
	at org.eclipse.jetty.servlet.ServletHandler$ChainEnd.doFilter(ServletHandler.java:1665) ~[?:?]

To Reproduce
Query discovery of AS

Expected behavior
Response must be returned successfully.
@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 self-assigned this Sep 29, 2022
yuriyz added a commit that referenced this issue Sep 29, 2022
@yuriyz
fix(jans-auth-server): npe in discovery if SSA endpoint is absent #2497
40c2380 
docs: no docs
#2497
@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): npe in discovery if SSA endpoint is absent #2497…
c3b00b4 
… (#2498)

docs: no docs
#2497
pujavs pushed a commit that referenced this issue Sep 30, 2022
@yuriyz @pujavs
fix(jans-auth-server): npe in discovery if SSA endpoint is absent #2497…
3e33f96 
… (#2498)

docs: no docs
#2497
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 <[email protected]>
Co-authored-by: Devrim <[email protected]>
Co-authored-by: Michael Schwartz <[email protected]>
@mo-auto mo-auto mentioned this issue Nov 1, 2022
Merged
This was referenced Nov 1, 2022
Merged
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
Development

Successfully merging a pull request may close this issue.

fix(jans-auth-server): npe in discovery if SSA endpoint is absent #2497 JanssenProject/jans
1 participant
@yuriyz