Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Integration - Python Backend #751

Merged
merged 378 commits into from
Jan 6, 2025
Merged

Integration - Python Backend #751

merged 378 commits into from
Jan 6, 2025

Conversation

aloftus23
Copy link
Contributor

@aloftus23 aloftus23 commented Jan 3, 2025 •
edited
Loading

🗣 Description

Complete the merge of the entire python backend into staging-cd.

This includes the entire API gateways, lambdas, and worker.

🧪 Testing

Added pytests and pylint checks that are all passing

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All future TODOs are captured in issues, which are referenced
    in code comments.
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated
    to reflect the changes in this PR.
  • Tests have been added and/or modified to cover the changes in this PR.
  • All new and existing tests pass.

✅ Pre-merge checklist

  • Revert dependencies to default branches.
  • Finalize version.

✅ Post-merge checklist

  • Create a release.

chrtorres and others added 30 commits October 29, 2024 16:11
@chrtorres
Saved Search tests skeleton
cdf9cfb
@aloftus23
Add stats endpoints,settings and management commands
b7756b2
@chrtorres
Code cleanup
aa217dc
@aloftus23
Add register endpoints and post User with tests in test_user.py
7195c90
@aloftus23
Merge pull request #710 from cisagov/CT-saved-search-tests
764e04f 
Saved Searches Test Base
@Matthew-Grayson
Update python user endpoints.
6e6d527
@Matthew-Grayson
Merge origin MG-python-serverless into remote MG-python-serverless.
12ec1aa
@Matthew-Grayson
Merge origin MG-python-serverless into remote MG-python-serverless.
0e43828
@Matthew-Grayson
Fix lint issues in email.py.
06771bc
@Matthew-Grayson
Merge branch 'AL-python-serverless' of github.com:cisagov/XFD into MG…
e525a9f 
…-python-serverless
@aloftus23
Add status endpoints
d17790b
@cduhn17
Merge branch 'AL-python-serverless' into cd_elasticache_updated
0bf960c
@aloftus23
Update env variables at and integrate at settings.py
d569f5a
@aloftus23
Merge branch 'cd_elasticache_updated' of github.com:cisagov/XFD into …
7535e27 
…cd_elasticache_updated
@nickviola
Remove Bearer requirement from Authorization token and update docstrings
9cca210
@nickviola
Update auth to use X-API-KEY or Authorization header and not require …
c9db522 
…both
@aloftus23
Update variable names for attribute errors
f0fe7b8
@Matthew-Grayson
Remove ReadySetCyber from Python Backend
0fe8b53 
Delete RSC models and endpoints.
Delete RSC user type.
Delete rscSync scan.
@nickviola
Update python auth function comments and spacing
b20fd0c
@nickviola
Update python auth function comments and spacing
1eb4584
@aloftus23
First commit to integration
5d0e558 
Includes:
- AL-python-serverless latest
-  .github updates from prod to integration
- terraform/serverless updates from prod to integration
@aloftus23
fix backend.yml
37bcedf
@aloftus23
Run linter and fix backend.yml
29434c3
@nickviola
Update auth header functionality to use either x-api-key or authoriza…
8a42c31 
…tion headers
@aloftus23
Fix the pytest configuration and add to makefile
b501721
@aloftus23
make sure docker is running for the pytests
060c3ff
@aloftus23
Delete the extra test_python and run pytest in "test"
5aed36f
@aloftus23
Fix backend.yml
be8b5e2
@aloftus23
Update infrastructure pipeline to use stag-cd instead of stage
c07443f
@JCantu248
Add test cases for domains and fixed issues in domains and vulnerabil…
bd20a76 
…ities.
@aloftus23 aloftus23 marked this pull request as ready for review January 6, 2025 20:42
@aloftus23 @github-advanced-security
Fix code scanning alert no. 208: Uncontrolled data used in path expre…
934f8f6 
…ssion

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@cduhn17 cduhn17 merged commit 2202eaf into develop Jan 6, 2025
22 of 33 checks passed
@cduhn17 cduhn17 deleted the integration branch January 6, 2025 20:56
@cduhn17 cduhn17 restored the integration branch January 6, 2025 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@schmelz21 schmelz21 schmelz21 approved these changes

@rapidray12 rapidray12 rapidray12 approved these changes

@dav3r dav3r Awaiting requested review from dav3r dav3r is a code owner

@felddy felddy Awaiting requested review from felddy felddy is a code owner

@jsf9k jsf9k Awaiting requested review from jsf9k jsf9k is a code owner

@mcdonnnj mcdonnnj Awaiting requested review from mcdonnnj mcdonnnj is a code owner

@cduhn17 cduhn17 Awaiting requested review from cduhn17 cduhn17 is a code owner

@Matthew-Grayson Matthew-Grayson Awaiting requested review from Matthew-Grayson Matthew-Grayson is a code owner

@nickviola nickviola Awaiting requested review from nickviola nickviola is a code owner

Assignees

@aloftus23 aloftus23

Labels
backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@aloftus23 @schmelz21 @rapidray12 @cduhn17 @chrtorres @Matthew-Grayson @nickviola @JCantu248