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.

Sign up for GitHub

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

Jump to bottom

Querying MPI by mhv uuid for mhv credential authentication on Sign in Service #18970

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Querying MPI by mhv uuid for mhv credential authentication on Sign in Service #18970

wants to merge 1 commit into from
+34 −34

Conversation

bosawt
Copy link
Contributor

@bosawt bosawt commented Oct 18, 2024

Summary

  • This PR reintroduces querying MPI by MHV UUID for Sign in Service authentication. This updates the MHV id being queried to be the 'production' value, whereas previously it was a value seemingly used solely in staging and below

The rest of the PR description is the same as this: #18862

The difference is just confirming the query will find a 200MH MHV UUID, rather than a 200MHS MHV UUID

@bosawt bosawt requested a review from a team as a code owner October 18, 2024 22:14
@va-vfs-bot va-vfs-bot temporarily deployed to sis_search_mpi_by_mhv/main/main October 18, 2024 22:16 Inactive
@va-vfs-bot va-vfs-bot temporarily deployed to sis_search_mpi_by_mhv/main/main October 18, 2024 22:34 Inactive
rileyanderson
rileyanderson previously approved these changes Oct 22, 2024
View reviewed changes
bramleyjl
bramleyjl previously approved these changes Oct 22, 2024
View reviewed changes
@bosawt bosawt dismissed stale reviews from bramleyjl and rileyanderson via 2db4add October 23, 2024 22:52
@va-vfs-bot va-vfs-bot temporarily deployed to sis_search_mpi_by_mhv/main/main October 23, 2024 22:53 Inactive
@platform-support-slack-integration platform-support-slack-integration bot mentioned this pull request Oct 24, 2024
Closed
@github-actions GitHub Actions
Copy link

Backend-review-group approval confirmed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rileyanderson rileyanderson rileyanderson approved these changes

@bramleyjl bramleyjl bramleyjl approved these changes

@stevenjcumming stevenjcumming stevenjcumming approved these changes

Assignees
No one assigned
Labels
require-backend-approval test-passing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@bosawt @rileyanderson @bramleyjl @stevenjcumming @va-vfs-bot