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

Add Input Validation for /patient API Search Parameters #2580

Closed
bodhish opened this issue Nov 5, 2024 · 2 comments · Fixed by #2588
Closed

Add Input Validation for /patient API Search Parameters #2580

bodhish opened this issue Nov 5, 2024 · 2 comments · Fixed by #2588
Assignees
@DraKen0009
Labels
Carried over

Comments

@bodhish
Copy link
Member

bodhish commented Nov 5, 2024

Update the /patient API to incorporate input validation on search parameters, ensuring user inputs conform to defined limits.

For instance, the name field currently accepts up to 200 characters, but the search functionality inadvertently allows more.

We should also verify the same for all other supported fields.

Also the api should be /patients instead of /patient right? 🤔
@DraKen0009
Copy link
Contributor

Can I pick this issue?

@himanshu-sharmav
Copy link

Hi, @bodhish @sainak I would like to work on this issue.

@github-project-automation github-project-automation bot moved this to Triage in Care Nov 5, 2024
@rithviknishad rithviknishad moved this from Triage to Up Next in Care Nov 5, 2024
@DraKen0009 DraKen0009 mentioned this issue Nov 9, 2024
Merged
2 tasks
@github-project-automation github-project-automation bot moved this from Up Next to Done in Care Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DraKen0009 DraKen0009

Labels
Carried over
Projects
Care
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add tests and validation for filterset DraKen0009/care
3 participants
@bodhish @DraKen0009 @himanshu-sharmav