Skip to content

New Scanner: AngularJS CSTI Scanner (closes #216)#248

Merged
J12934 merged 15 commits intomainfrom
scanner/acstis
Mar 8, 2021
Merged

New Scanner: AngularJS CSTI Scanner (closes #216)#248
J12934 merged 15 commits intomainfrom
scanner/acstis

Conversation

@paulschmelzer
Copy link
Copy Markdown
Contributor

@paulschmelzer paulschmelzer commented Dec 14, 2020

Integrates the AngluarJS Client-Side Template Injection Scanner.

Because the scanner provides some request option by extending a python startup script, I had to inject these options before the script is started. Therefore the user might provide a config map with a config file containing the request options according to the README.

If you like to test the vulnerabiltity run the following app:
sudo docker run --rm -ti -p 127.0.0.1:5000:5000 blabla1337/owasp-skf-lab:csti
Because acstis can't determine the AngularJS version you have to add the flag -av 1.5.0 to the scanner. It is very strange that the scanner can not guess the version because it is very obvioulsy imported in the sites HTML.

closes #248

@paulschmelzer paulschmelzer added the scanner Implement or update a security scanner label Dec 14, 2020
@paulschmelzer paulschmelzer requested a review from J12934 December 14, 2020 14:53
@paulschmelzer paulschmelzer linked an issue Dec 14, 2020 that may be closed by this pull request
8 tasks
@paulschmelzer paulschmelzer marked this pull request as ready for review December 21, 2020 12:46
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Copy link
Copy Markdown
Member

@nigthknight nigthknight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great to me but I have one question. Why is it necessary to save the acstis-script.py in the secureCodeBox repository?

J12934 and others added 7 commits February 3, 2021 08:59
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
@nigthknight nigthknight enabled auto-merge March 8, 2021 09:00
@J12934 J12934 disabled auto-merge March 8, 2021 16:02
@J12934 J12934 merged commit fab01bd into main Mar 8, 2021
@J12934 J12934 deleted the scanner/acstis branch March 8, 2021 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scanner Implement or update a security scanner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🚓 Integrate a new Angular Client-Side Template Injection Scanner

3 participants