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

Enhancement: surface parse errors more comprehensively throughout the AF #131

Open
davidjwbbc opened this issue Jan 23, 2024 · 0 comments
Open
Labels
enhancement New feature or request

Comments

@davidjwbbc
Copy link
Contributor

Description

When the OpenAPI code templates were updated we introduced an "out" parameter which returns the parse error reason string. Where the OpenAPI model interfaces are directly used by the M1/M5 API this reason is now reported, but where the parsing is buried within other functions (dynamic-policy, network-assistance-session, etc.) this error reason is not surfaced back to the API response.

Design

Add a const char **parse_err parameter to all functions which perform any parsing or validation of JSON to return any parse error reason strings back up to the API handling so that it can be returned as part of the ProblemDetails description.

@davidjwbbc davidjwbbc added the enhancement New feature or request label Jan 23, 2024
@davidjwbbc davidjwbbc assigned davidjwbbc and unassigned davidjwbbc Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Backlog
Development

No branches or pull requests

1 participant