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.

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

#627 Remove readOnly properties from request instead of throwing Bad Request errors #753

Closed
wants to merge 1 commit into from
Closed

#627 Remove readOnly properties from request instead of throwing Bad Request errors #753

wants to merge 1 commit into from

Conversation

pilerou
Copy link
Contributor

@pilerou pilerou commented Aug 23, 2022

As discussed in #627 issue, whe should remove readonly data from request on validation.
Therefore, routes will receive a modified request wihtout readonly proprerties.

In this pull request :

  • AJV integration for readonly fields has been changed. In the same time,
  • Unit tests has been modified according to the new mescanism

@pilerou pilerou closed this Jan 30, 2024
@pilerou
Copy link
Contributor Author

pilerou commented Jan 30, 2024

I close this PR as a new one had been created #895

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pilerou