Skip to content
This repository has been archived by the owner on Dec 31, 2021. It is now read-only.

fix(JSON): Resolve mapping issues #44

Merged

Conversation

hypery2k
Copy link
Contributor

@hypery2k hypery2k commented Apr 9, 2020

fixes #43

@flytreeleft flytreeleft merged commit b3c0158 into flytreeleft:master Apr 11, 2020
@flytreeleft
Copy link
Owner

flytreeleft commented Apr 11, 2020

@hypery2k The error Unrecognized field "xxx" should be disappeared if we use the latest version of the Keycloak SDK.

But, if we disable the DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES explicitly (JsonSerialization.mapper.disable(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES)), maybe the plugin can be compatible with the latest Keycloak without upgrading the Keycloak SDK anymore. So, I think the PR is worthy of merging.

Thanks @hypery2k :).

@hypery2k
Copy link
Contributor Author

That was the idea behind it.
I compiled it with the older Keycloak SDK and tried it within a Keycloak 9 and seems to works.

@flytreeleft
Copy link
Owner

@hypery2k Great, thanks for your hard work :).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error executing http method durring Authorization
2 participants