Skip to content

Conversation

coopernetes
Copy link
Contributor

Closes #690

  • allow for the setting of configuration for MongoDB database
    connection string as well as cookie secret via environment
    variables as an alternative to JSON-based configuration

Useful in environments that split config between secrets (ie.
stored in a secret manager) and plain app configurations such
as commitConfig, attestation, etc. that may not be considered
sensitive & wouldn't be appropriate in a secret manager.

- allow for the setting of configuration for MongoDB database
  connection string as well as cookie secret via environment
  variables as an alternative to JSON-based configuration

Useful in environments that split config between secrets (ie.
stored in a secret manager) and plain app configurations such
as commitConfig, attestation, etc. that may not be considered
sensitive & wouldn't be appropriate in a secret manager.
Copy link

netlify bot commented Jul 3, 2025

Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Name Link
🔨 Latest commit a007613
🔍 Latest deploy log https://app.netlify.com/projects/endearing-brigadeiros-63f9d0/deploys/686796dd831fc20008f0c836

Copy link

codecov bot commented Jul 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.91%. Comparing base (f96a8c4) to head (a007613).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1086      +/-   ##
==========================================
+ Coverage   76.87%   76.91%   +0.04%     
==========================================
  Files          55       55              
  Lines        2266     2270       +4     
  Branches      252      255       +3     
==========================================
+ Hits         1742     1746       +4     
  Misses        494      494              
  Partials       30       30              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@JamieSlome JamieSlome left a comment

Choose a reason for hiding this comment

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

LGTM! 🍰

@JamieSlome JamieSlome enabled auto-merge July 4, 2025 08:54
@JamieSlome JamieSlome merged commit 53fded7 into finos:main Jul 4, 2025
14 checks passed
@coopernetes coopernetes deleted the feat/mongo-env branch August 21, 2025 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

making COOKIE_SECRET an env var
2 participants