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

feat(iatlas): migrate iAtlas GraphQL API to the monorepo #2559

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

feat(iatlas): migrate iAtlas GraphQL API to the monorepo #2559

wants to merge 2 commits into from

Conversation

tschaffter
Copy link
Member

@tschaffter tschaffter commented Mar 11, 2024 •
edited
Loading

Closes #2558

Description

Migrate iAtlas-api to this monopore.

Changelog

  • create the project iatlas-api
  • migrate the code required to build the Docker image
  • create Docker Compose file for the new project

Suggested Future Work

  • migrate Python dependencies from requirements.txt and requirements-dev.txt to Poetry

Preview

Build iAtlas API image

nx build-image iatlas-api

Build all the iAtlas images if you want to start the API dependencies too:

iatlas-build-images

Output:

$ iatlas-build-images 
 
    ✔  nx run iatlas-postgres:build-image (20s)
    ✔  nx run iatlas-api:build-image (42s)
    ✔  nx run iatlas-data:build-image (1m)

 ——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  NX   Successfully ran target build-image for 3 projects (1m)

Start iAtlas API with Docker Compose

nx serve-detach iatlas-api

@tschaffter tschaffter self-assigned this Mar 11, 2024
@tschaffter @andrewelamb
add python code
9ce7706 
Co-authored-by: andrewelamb <[email protected]>
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 11, 2024
View reviewed changes
apps/iatlas/api/requirements.txt
@@ -0,0 +1,13 @@
ariadne==0.13.0

Check failure

Code scanning / Trivy

flask: Possible disclosure of permanent session cookie due to missing Vary: Cookie header High

Package: Flask
Installed Version: 1.1.2
Vulnerability CVE-2023-30861
Severity: HIGH
Fixed Version: 2.3.2, 2.2.5
Link: CVE-2023-30861
apps/iatlas/api/requirements.txt
@@ -0,0 +1,13 @@
ariadne==0.13.0

Check failure

Code scanning / Trivy

python-werkzeug: high resource usage when parsing multipart form data with many fields High

Package: Werkzeug
Installed Version: 1.0.1
Vulnerability CVE-2023-25577
Severity: HIGH
Fixed Version: 2.2.3
Link: CVE-2023-25577
apps/iatlas/api/requirements.txt
@@ -0,0 +1,13 @@
ariadne==0.13.0

Check failure

Code scanning / Trivy

There MultipartParser usage in Encode&#39;s Starlette python framework bef ... High

Package: starlette
Installed Version: 0.13.4
Vulnerability CVE-2023-30798
Severity: HIGH
Fixed Version: 0.25.0
Link: CVE-2023-30798
apps/iatlas/api/requirements.txt
@@ -0,0 +1,13 @@
ariadne==0.13.0

Check failure

Code scanning / Trivy

`python-multipart` is a streaming multipart parser for Python. When us ... High

Package: starlette
Installed Version: 0.13.4
Vulnerability CVE-2024-24762
Severity: HIGH
Fixed Version: 0.36.2
Link: CVE-2024-24762
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tschaffter tschaffter

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Story] Migrate iAtlas GraphQL API to the monorepo
1 participant
@tschaffter