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: #885 fix admin management db username in flyway #1051

Merged
merged 18 commits into from
Nov 24, 2023

Conversation

MCatherine1994
Copy link
Contributor

@MCatherine1994 MCatherine1994 commented Nov 24, 2023

Solve the error in pipeline when flyway when run v33. Because in v32, we already created the new db user "fam_proxy_admin_management_api", but later on we changed the username to "fam_admin_management_api". So when run v33, it can't find the user "fam_admin_management_api", because last time we created the user called "fam_proxy_admin_management_api". So change the db username back.

…or team discussion, add router guard for checking permission and not allow self grant, refs: #885
Copy link

sonarcloud bot commented Nov 24, 2023

[nr-forests-access-management_admin] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@MCatherine1994 MCatherine1994 merged commit bb5352c into main Nov 24, 2023
14 checks passed
@MCatherine1994 MCatherine1994 deleted the feat/885-add-admin-endpoint branch November 24, 2023 01:09
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