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.

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

fix(jans-config-api): OpenID Connect Client displayName update issue #1724

Closed
pujavs opened this issue Jul 6, 2022 · 3 comments
Closed

fix(jans-config-api): OpenID Connect Client displayName update issue #1724

pujavs opened this issue Jul 6, 2022 · 3 comments
Assignees
Labels
comp-jans-config-api Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Milestone

Comments

@pujavs
Copy link
Contributor

pujavs commented Jul 6, 2022

Describe the bug
Error when updating client displayName

com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot deserialize value of type java.lang.String from Object value (token JsonToken.START_OBJECT)
at [Source: (org.eclipse.jetty.server.HttpInput); line: 1, column: 320] (through reference chain: io.jans.as.common.model.registration.Client["clientName"])

To Reproduce
Steps to reproduce the behaviour:

  1. Execute jans-cli -> /opt/jans/jans-cli/config-cli.py -use-test-client
  2. Go to option 8(OAuth) -> 1(OpenID Connect - Clients) -> 6(Update modified properties of OpenId Connect client by Inum)
  3. Key in inum and update displayName
  4. See error
    com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot deserialize value of type java.lang.String from Object value (token JsonToken.START_OBJECT)
    at [Source: (org.eclipse.jetty.server.HttpInput); line: 1, column: 320] (through reference chain: io.jans.as.common.model.registration.Client["clientName"])

Expected behavior
Client displayName should be updated

Screenshots
image

Logs:
image

@mo-auto mo-auto added comp-jans-config-api Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Jul 6, 2022
@moabu moabu added this to the 1.0.2 milestone Jul 8, 2022
@moabu moabu modified the milestones: 1.0.2, 1.0.3 Aug 30, 2022
@devrimyatar
Copy link
Contributor

@pujavs Still having issue:

21-09 15:16:55.753 DEBUG configapi.core.util.Jackson Jackson.java:53- Patch details - patchAsString:[{"op": "replace", "path": "/displayName", "value": "My Test Client"}], obj:DeletableEntity{expirationDate=null, deletable=false} BaseEntry [dn=inum=9abe8fc2-4efd-4312-b14b-1b8ef50e1c0e,ou=clients,o=jans]
2022-09-21 15:16:55.883:WARN :oejs.HttpChannel:qtp1793436274-17: handleException /jans-config-api/api/v1/openid/clients/9abe8fc2-4efd-4312-b14b-1b8ef50e1c0e com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot deserialize value of type `java.lang.String` from Object value (token `JsonToken.START_OBJECT`)| at [Source: UNKNOWN; byte offset: #UNKNOWN] (through reference chain: io.jans.as.common.model.registration.Client["clientName"])

@devrimyatar
Copy link
Contributor

Working (MySQL backend)

vokoscreenNG-2023-01-20_13-40-53.mp4

@pujavs
Copy link
Contributor Author

pujavs commented Jan 20, 2023

closing this issue as successfully tested by @devrimyatar

@pujavs pujavs closed this as completed Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-config-api Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Development

No branches or pull requests

5 participants