Skip to content

Merge Testing #81

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.

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

Merged
merged 16 commits into from
May 16, 2025
Merged

Merge Testing #81

merged 16 commits into from
May 16, 2025

Conversation

bitbacchus
Copy link
Member

No description provided.

esthako and others added 16 commits January 31, 2025 17:08
Design: studentsCanSeeStatistics: true
studentsCanSeeStatistic: true
Revert "studentsCanSeeStatistic: true"
…r-compose for production serving. Fixed bugs that only occured when running inside docker.
This reverts commit 11d1209.
…ed docker-compose for production serving. Fixed bugs that only occured when running inside docker."

This reverts commit 1d32821.
* Added dockerfiles for frontend and backend containers. Adjusted docker-compose for production serving. Fixed bugs that only occured when running inside docker.

* Added missing entry MONGODB_HOST

* Create .env

* Update .env.template

* Delete packages/frontend/.env

The variables are now in docker/.env-production

* Update docker-compose.prod.yaml

Typo (I think...)

---------

Co-authored-by: Hendrik Belitz <[email protected]>
Removes hard-coded internal API key
Container.set<string[]>("api-keys", process.env.API_KEYS?.split(",") ?? []);
Container.set<string[]>(
"api-keys",
process.env.API_KEYS?.split(",") ?? ["25868755-c11c-42ee-b4ed-2115ac982ba4"]

Check failure

Code scanning / CodeQL

Hard-coded credentials Critical

The hard-coded value "25868755-c11c-42ee-b4ed-2115ac982ba4" is used as
authorization header
.
@bitbacchus bitbacchus merged commit dd4b48e into main May 16, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants