Skip to content

feat: Add Google Cloud Identity Platform authenticator [2/4]#1123

Merged
jcscottiii merged 1 commit intomainfrom
jcscottiii/gcip_authenticator
Feb 4, 2025
Merged

feat: Add Google Cloud Identity Platform authenticator [2/4]#1123
jcscottiii merged 1 commit intomainfrom
jcscottiii/gcip_authenticator

Conversation

@jcscottiii
Copy link
Collaborator

@jcscottiii jcscottiii commented Feb 2, 2025

Background: In a future PR, we will introduce an authenticator http middleware that will call Authenticate.

This change is the Google Cloud Identity Platform (GCIP) implementation of that Authenticate interface.

GCIP uses firebase under the hood. This implementation comes straight from the admin SDK docs: https://firebase.google.com/docs/auth/admin/verify-id-tokens#go

This change also adds a new User type that the middleware will expect.

Other changes:

  • Run go mod tidy on each package via make go-tidy - Needed after importing firebaseauth.Token into lib

This is a split up of #1118

@jcscottiii jcscottiii force-pushed the jcscottiii/gcip_authenticator branch from ceb21be to 048cb0e Compare February 2, 2025 16:58
@jcscottiii jcscottiii changed the title feat: Add Google Cloud Identity Platform authenticator feat: Add Google Cloud Identity Platform authenticator [2/4] Feb 3, 2025
Base automatically changed from jcscottiii/add-placeholder-routes-for-saved-search to main February 4, 2025 14:01
Background: In a future PR, we will introduce an authenticator http middleware that
will call Authenticate.

This change is the Google Cloud Identity Platform (GCIP) implementation of that Authenticate interface.

GCIP uses firebase under the hood. This implementation comes straight from the admin SDK docs:
https://firebase.google.com/docs/auth/admin/verify-id-tokens#go

This change also adds a new User type that the middleware will expect.

Other changes:
- Run go mod tidy on each package via `make go-tidy` - Needed after importing `firebaseauth.Token` into `lib`
@jcscottiii jcscottiii force-pushed the jcscottiii/gcip_authenticator branch from 048cb0e to 77c4af8 Compare February 4, 2025 14:09
@jcscottiii jcscottiii enabled auto-merge February 4, 2025 14:09
@jcscottiii jcscottiii added this pull request to the merge queue Feb 4, 2025
Merged via the queue into main with commit 528738a Feb 4, 2025
6 checks passed
@jcscottiii jcscottiii deleted the jcscottiii/gcip_authenticator branch February 4, 2025 14:48
@DanielRyanSmith DanielRyanSmith mentioned this pull request Feb 5, 2025
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