augustin-wien / augustina-backend

An open-source web shop designed for selling magazines on the street.

GNU Affero General Public License v3.0

7 stars 0 forks source link

Add security check for duplicate items #111

Closed lebe1 closed 12 months ago

lebe1 commented 1 year ago

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Description

IMPORTANT TO KNOW

Please check added TODO comment if this line and the whole user variable in POST-Request is unnecessary due to Keycloak User header

CHANGES

Adds security check to input only one item once
Adds test for security check
Found possible unused variable

TODO

Checklist:

[x] I have commented my code (or ChatGPT did), particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings, neither in my IDE nor in my browser
[x] I have added tests that prove my fix is effective or that my feature works