Skip to content

Feat/get one ssn #535

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

Merged
merged 12 commits into from
Feb 13, 2025
Merged

Conversation

jusdino
Copy link
Collaborator

@jusdino jusdino commented Feb 10, 2025

Description List

  • Added a GET .../providers/:providerId/ssn endpoint to retrieve a provider's SSN
  • Added a new readSSN permissions action for compact/jurisdiction-level access to SSNs
  • Added basic metrics/Alarms for monitoring activity on this sensitive endpoint

Closes #392

Copy link
Collaborator

@landonshumway-ia landonshumway-ia left a comment

Choose a reason for hiding this comment

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

Great, this looks much better with the CCPermissionsAction enum

@landonshumway-ia
Copy link
Collaborator

@jlkravitz This should be ready for your review.

@jusdino jusdino mentioned this pull request Feb 11, 2025
Copy link
Collaborator

@jlkravitz jlkravitz left a comment

Choose a reason for hiding this comment

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

Looks good! Appreciate the attention to detail when it comes to SSN access. A few comments/questions.

jlkravitz
jlkravitz previously approved these changes Feb 13, 2025
@jlkravitz
Copy link
Collaborator

@isabeleliassen This is good to merge. The failed test is due to an out-of-date dependency that has been fixed in development, but not in this branch.

@isabeleliassen
Copy link
Collaborator

@jlkravitz we'll need your approval review as well!

@isabeleliassen isabeleliassen self-requested a review February 13, 2025 18:19
@isabeleliassen isabeleliassen merged commit 1031f65 into csg-org:development Feb 13, 2025
3 checks passed
@jusdino jusdino deleted the feat/get-one-ssn branch February 13, 2025 20:21
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.

Add new get-one full SSN endpoint (backend)
4 participants