Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix API endpoint permission for the "AttachmentMixin" class #3218

Merged

Conversation

SchrodingersGat
Copy link
Member

@SchrodingersGat SchrodingersGat commented Jun 18, 2022

  • Any authenticated user could perform CREATE and UPDATE operations on attachments
  • Could be performed via the browsable DRF API
  • Could also be performed via the front-end (with some advanced jiggering of OPTIONS code)

- Any authenticated user could perform CREATE and UPDATE operations on attachments
- Could be performed via the browsable DRF API
- Could also be performed via the front-end (with some advaned jiggering of OPTIONS code)
@SchrodingersGat SchrodingersGat added bug Identifies a bug which needs to be addressed security Relates to a security issue labels Jun 18, 2022
@SchrodingersGat SchrodingersGat added this to the 0.8.0 milestone Jun 18, 2022
@SchrodingersGat SchrodingersGat added the api Relates to the API label Jun 18, 2022
@SchrodingersGat SchrodingersGat merged commit 12fcccb into inventree:master Jun 18, 2022
@SchrodingersGat SchrodingersGat deleted the attachment-permission-fix branch June 18, 2022 04:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
api Relates to the API bug Identifies a bug which needs to be addressed security Relates to a security issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants