secDevLabs icon indicating copy to clipboard operation
secDevLabs copied to clipboard

[A1] - Broken Access Control - Camplake-API

Open vitor-mauricio opened this issue 3 years ago • 1 comments

This solution refers to which of the apps?

[A1] - Broken Access Control - Camplake-API

What did you do to mitigate the vulnerability?

Adding signature validation to incoming user`s token.

Did you test your changes? What commands did you run?

The attack narrative did not working anymore.

vitor-mauricio avatar Jun 07 '22 19:06 vitor-mauricio

This pull request introduces 2 alerts when merging c668fa0e0ee9e2e0678efd30d8cc59211fd8dfe4 into 46a89c9f83781ddc61daf6dd46d6811f509e155b - view on LGTM.com

new alerts:

  • 2 for Missing error check

lgtm-com[bot] avatar Jun 07 '22 19:06 lgtm-com[bot]

Hey @vitor-mauricio you fix the vulnerability for this exercise. @InesCardinot give you an alert about hardcoded credentials, this won't be part of the exercise, but if you know how to fix that two, I'll give you a bonus point! 😄

fguisso avatar Nov 09 '22 14:11 fguisso