Skip to content

Flask analysis#2

Merged
TheTeaCat merged 47 commits intodevfrom
flask-analysis
May 31, 2023
Merged

Flask analysis#2
TheTeaCat merged 47 commits intodevfrom
flask-analysis

Conversation

@TheTeaCat
Copy link
Member

@TheTeaCat TheTeaCat commented May 23, 2023 •
edited
Loading

  • Restructured this to make it a bit more extensible
  • Added precommit
  • Tidied the Dockerfile up a bit
  • Initial implementation of static analysis for Flask projects using ast to parse .py files

TheTeaCat added 23 commits May 22, 2023 11:07
@TheTeaCat
Udpate REQUEST_SESSION
b6e32e1
@TheTeaCat
Extract consts and env vars to consts.py and env.py
2476a25
@TheTeaCat
Tidy dockerfile
6cf297c
@TheTeaCat
Add precommit
fadfe97
@TheTeaCat
Initial run of precommit on all files
258303c
@TheTeaCat
Extract openapi validation to openapi submodule
a31a839
@TheTeaCat
Extract credentials utils to credentials submodule
0723eee
@TheTeaCat
Remove unused utils
eeb92a8
@TheTeaCat
Refactor list_orgs_for_user to get_repositories_to_scan
054d91c
@TheTeaCat
Refactor process_repo -> scan_repository using pygithub objects
87885f9
@TheTeaCat
Move python static analysis into static_analysis submodule
2a0c10a
@TheTeaCat
Add missing type hints
d39e746
@TheTeaCat
Setup main to scan with token
805cb4f
@TheTeaCat
Fix undefined local var in openapi validation
2a67177
@TheTeaCat
Fix respect_rate_limit for methods which return None
a0d1bd5
@TheTeaCat
Add some debug logging
e943170
@TheTeaCat
Fix typo
04f5533
@TheTeaCat
Extract file scanning to func
425bd60
@TheTeaCat
Remove unused consts
b620391
@TheTeaCat
Use ast to find imports
48bc680
@TheTeaCat
Add flask helloworld example test
3f44148
@TheTeaCat
Remove outdated test
5fd13bc
@TheTeaCat
Add setup.cfg with pytest & flake8 config
f22e4b4
@TheTeaCat TheTeaCat self-assigned this May 23, 2023
@TheTeaCat TheTeaCat mentioned this pull request May 23, 2023
Closed
@TheTeaCat TheTeaCat marked this pull request as ready for review May 24, 2023 11:59
@TheTeaCat TheTeaCat requested a review from finnci May 24, 2023 12:51
finnci
finnci reviewed May 24, 2023
View reviewed changes
@TheTeaCat TheTeaCat requested a review from finnci May 31, 2023 14:33
@TheTeaCat TheTeaCat mentioned this pull request May 31, 2023
Merged
@TheTeaCat TheTeaCat merged commit f2cc329 into dev May 31, 2023
@TheTeaCat TheTeaCat deleted the flask-analysis branch June 6, 2023 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@finnci finnci finnci approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@TheTeaCat TheTeaCat

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TheTeaCat @finnci