-
Notifications
You must be signed in to change notification settings - Fork 24
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
Fixes #23775: Migrate api-authorizations, auth-backends to zio-json #612
Fixes #23775: Migrate api-authorizations, auth-backends to zio-json #612
Conversation
...izations/src/main/scala/com/normation/plugins/apiauthorizations/UserTokenApiDefinition.scala
Show resolved
Hide resolved
api-authorizations/src/test/scala/com/normation/plugins/apiauthorizations/api/UserApiTest.scala
Outdated
Show resolved
Hide resolved
...izations/src/main/scala/com/normation/plugins/apiauthorizations/UserTokenApiDefinition.scala
Outdated
Show resolved
Hide resolved
...izations/src/main/scala/com/normation/plugins/apiauthorizations/UserTokenApiDefinition.scala
Outdated
Show resolved
Hide resolved
api-authorizations/src/test/resources/authorizations_api/api_users.yml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some little things to change, but that's a super nive evolution in our code base, GG.
It can be merged once the remark are addressed.
PR updated with a new commit |
3b15d10
to
e931686
Compare
This PR is not mergeable to upper versions. |
OK, squash merging this PR |
e931686
to
120deab
Compare
https://issues.rudder.io/issues/23775
Migrating api-authorizations, there needed to be a rewrite of the lift-json serializer into a data transfer case class and the added tests ensures that the display/hiding of the token value is the same after migrating to zio-json...