firebase-admin-go icon indicating copy to clipboard operation
firebase-admin-go copied to clipboard

FR: Allow for Auth Provider Configs to be set

Open ToeFungi opened this issue 4 years ago • 7 comments

At the moment it appears that only OIDC and SAML providers can be set via the Go client. Would be incredibly useful to add Firebase auth providers such as phone and email/password so that we can programmatically manage this with something like Terraform.

API

https://cloud.google.com/identity-platform/docs/reference/rest/v2/projects/updateConfig

Note

It appears that the Node SDK has this functionality available already.

ToeFungi avatar May 21 '21 11:05 ToeFungi

I found a few problems with this issue:

  • I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.
  • This issue does not seem to follow the issue template. Make sure you provide all the required information.

google-oss-bot avatar May 21 '21 11:05 google-oss-bot

I think this was already implemented in #344

https://pkg.go.dev/firebase.google.com/go/v4/auth#UserToUpdate.ProviderToLink

@ToeFungi can you check if this meets your needs?

hiranya911 avatar May 21 '21 22:05 hiranya911

@hiranya911 if I'm understanding the documentation correctly, that enables the provider against a specific user?

We're looking to enable Phone and Email/Password auth providers at the project level. I'm not sure if that implementation handles project level enabling of the providers?

ToeFungi avatar May 26 '21 11:05 ToeFungi

Ah, that's more of a project management operation.

@bojeil-google wdyt?

hiranya911 avatar May 27 '21 22:05 hiranya911

I think this is part of bigger project to configure built-in providers (email/password, email link, phone, social providers, etc) at project level as well as tenant level. We have been wanting to do this for a while but this needs product approval.

bojeil-google avatar May 27 '21 22:05 bojeil-google

We created a custom client and Terraform provider to accomplish this as well as #441 in the mean time. Would be ideal if this was added officially though.

ToeFungi avatar May 28 '21 14:05 ToeFungi

Customer here. Not sure why the go client isn't comprehensive of the REST API.

numbsafari avatar Mar 13 '24 14:03 numbsafari