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

modified sanctuary db to use different credentials #120

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

MadelaineJ
Copy link
Collaborator

Problem
sanctuary db was still using the same credentials as medical

Solution

  • modified db init script to create sanctuary user (from env vars) with access to sanctuary db
  • modified database.py to use make sanctuary url use sanctuary credentials Note: should hopefully fix production login issues

Ticket URL
NA

Documentation
NA

Tests Run

  • make testapi
  • manually tested that loging in to sanctuary works

Problem
sanctuary db was still using the same credentials as medical

Solution
- modified db init script to create sanctuary user (from env vars) with access to sanctuary db
- modified database.py to use make sanctuary url use sanctuary credentials
Note: should hopefully fix production login issues

Ticket URL
NA

Documentation
NA

Tests Run
- make testapi
- manually tested that loging in to sanctuary works
@MadelaineJ MadelaineJ requested a review from critch646 March 4, 2024 22:41
Copy link
Collaborator

@critch646 critch646 left a comment

Choose a reason for hiding this comment

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

Full send

@MadelaineJ MadelaineJ merged commit d8cf673 into main Mar 4, 2024
1 check passed
@MadelaineJ MadelaineJ deleted the madelaine-sanctuary branch March 4, 2024 23:07
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.

2 participants