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

payload validation #187

Closed
wants to merge 8 commits into from
Closed

payload validation #187

wants to merge 8 commits into from

Conversation

sebm253
Copy link
Member

@sebm253 sebm253 commented Aug 15, 2022

No description provided.

@sebm253 sebm253 added this to the v1.0.0 milestone Aug 15, 2022
@sebm253 sebm253 linked an issue Aug 15, 2022 that may be closed by this pull request
@topi314 topi314 closed this Apr 10, 2023
@Malix-Labs
Copy link
Contributor

@topi314 is there any log as of why this PR didn't make it?

@topi314
Copy link
Member

topi314 commented Mar 1, 2024

client side payload validation would be a nice thing to have since this would stop bots making wrong requests

but we ultimately decided against it since its a lot of work to get it initially right and no one had the time for it

@Malix-Labs
Copy link
Contributor

@topi314 Understandable.
Do that mean that #44 should be closed as not planned?

@topi314
Copy link
Member

topi314 commented Mar 1, 2024

maybe, maybe not
its still something I'd like to have so I kept it open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

validation of payloads on rest routes
3 participants