8000 feat(jans-cedarling): Improve error message if json parsing of policy store failed · Issue #11216 · JanssenProject/jans · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(jans-cedarling): Improve error message if json parsing of policy store failed #11216

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
Open
olehbozhok opened this issue Apr 11, 2025 · 0 comments
Open

feat(jans-cedarling): Improve error message if json parsing of policy store failed #11216

olehbozhok opened this issue Apr 11, 2025 · 0 comments
Assignees
@olehbozhok @rmarinn
Labels
comp-jans-cedarling Touching folder /jans-cedarling enhancement kind-feature Issue or PR is a new feature request
Milestone
next-release

Comments

@olehbozhok
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Iimprove error message if json parsing of policy store failed.

Image

Describe the solution you'd like
It looks like the "missing ..." message is an error from the serde library. We don’t have full control over this message. However, it’s possible to implement a custom serde::Serializer trait implementation for each type we use. This would require significant effort but would give us more control over the returned error messages.

https://serde.rs/impl-serializer.html

Describe alternatives you've considered

Additional context
@olehbozhok olehbozhok added comp-jans-cedarling Touching folder /jans-cedarling enhancement labels Apr 11, 2025
@mo-auto mo-auto added the kind-feature Issue or PR is a new feature request label Apr 11, 2025
@moabu moabu added this to the next-release milestone Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@olehbozhok olehbozhok

@rmarinn rmarinn

Labels
comp-jans-cedarling Touching folder /jans-cedarling enhancement kind-feature Issue or PR is a new feature request
Projects
None yet
Milestone
next-release
Development

No branches or pull requests
4 participants
@olehbozhok @rmarinn @moabu @mo-auto
0