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

Change error message for /api/students/<int:id>/points GET #172

Open
MartaSien opened this issue Sep 7, 2024 · 1 comment · May be fixed by #209
Open

Change error message for /api/students/<int:id>/points GET #172

MartaSien opened this issue Sep 7, 2024 · 1 comment · May be fixed by #209
Assignees
@arkjedrz
Labels
enhancement New feature or request

Comments

@MartaSien
Copy link
Contributor

Error message 403 should be switched to 404 for /api/students/<int:id>/points when invalid id, which is not in a database, is given, as it may expose internal meta data.
@MartaSien MartaSien added the enhancement New feature or request label Sep 7, 2024
@arkjedrz
Copy link
Contributor

arkjedrz commented Sep 9, 2024

This should be done for all endpoints that might return 403.

@arkjedrz arkjedrz self-assigned this Sep 18, 2024
This was referenced Sep 25, 2024
Open
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arkjedrz arkjedrz

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Respond with 404 on not found and permission denied zetonteam/zeton_django
2 participants
@arkjedrz @MartaSien