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.

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

Fix/missing secret env #402

Merged
merged 3 commits into from
Nov 11, 2024
Merged

Fix/missing secret env #402

merged 3 commits into from
Nov 11, 2024

Conversation

MugemaneBertin2001
Copy link
Contributor

@MugemaneBertin2001 MugemaneBertin2001 commented Oct 30, 2024

PR Description

This fix adds missing SECRET variable in env during building docker image.

@codecov-commenter
Copy link

codecov-commenter commented Oct 30, 2024

Codecov Report

Attention: Patch coverage is 38.88889% with 11 lines in your changes missing coverage. Please review.

Please upload report for BASE (develop@3126013). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/resolvers/cohort.resolvers.ts 0.00% 4 Missing ⚠️
src/resolvers/eventResolver.ts 0.00% 2 Missing ⚠️
src/resolvers/ratingsResolvers.ts 0.00% 2 Missing ⚠️
src/helpers/organization.helper.ts 0.00% 1 Missing ⚠️
src/helpers/sendEventEmails.ts 0.00% 1 Missing ⚠️
src/resolvers/resolver.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #402   +/-   ##
==========================================
  Coverage           ?   16.58%           
==========================================
  Files              ?       89           
  Lines              ?     3558           
  Branches           ?     1019           
==========================================
  Hits               ?      590           
  Misses             ?     2959           
  Partials           ?        9           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@ceelogre ceelogre left a comment

Choose a reason for hiding this comment

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

The change is only needed in one middle for jwt.

@MugemaneBertin2001
Copy link
Contributor Author

MugemaneBertin2001 commented Oct 30, 2024

The required change is only needed in one middleware for JWT.

@ceelogre, please take another look and try the beta.

After removing const secret = (process.env.SECRET as string) || 'mysq_unique_secret' in the files you requested, the authentication didn't work as expected. I decided to restore those lines, and now the beta is built and ready for testing as usual.

You can review it at beta.devpulse.org. Once you've reviewed and approved, I'll remove the following section before merging:

pull_request:
  branches:
    - develop

@Salim-54
Copy link

Salim-54 commented Nov 1, 2024

Resolve the conflicts

@ceelogre ceelogre temporarily deployed to fix/missing-secret-env - devpulse-backend PR #402 November 11, 2024 15:37 — with Render Destroyed
@ceelogre ceelogre merged commit ba775e7 into develop Nov 11, 2024
2 checks passed
@ceelogre ceelogre deleted the fix/missing-secret-env branch November 11, 2024 15:52
elijahladdie pushed a commit that referenced this pull request Nov 11, 2024
* Added missing secret in env

* test: deployment

* feat: run the action on push

---------

Co-authored-by: ceelogre <[email protected]>
elijahladdie pushed a commit that referenced this pull request Nov 12, 2024
* Added missing secret in env

* test: deployment

* feat: run the action on push

---------

Co-authored-by: ceelogre <[email protected]>
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.

6 participants