firebase / firebase/firebase-admin-node

[firebase-admin] dynamic custom link suffix not added in email verification links in auth admin

Ouverte
#1,556 4 commentaires 1 réaction 0 personnes assignées Voir sur GitHub
api: auth
Langage dominant
TypeScript
Étoiles
1.7k
Forks
419
Merge moyen
3 j 10 h
PR mergées (30 j)
16

Description

* Operating System version: -
* Firebase SDK version: -
* Firebase Product: firebase-admin
* Node.js version: 15.9.0
* NPM version: 7.5.3

### [REQUIRED] Step 3:

#### Steps to reproduce:

link-suffix is not added in the generatePasswordResetLink and generateEmailVerificationLink of auth admin (firebase-admin/lib/auth) like https://firebase.google.com/docs/dynamic-links/custom-domains

it should be automatically added or ActionCodeSettings should include such param, for instance if you have example.com/link dynamic link, example.com/?link=... is generated instead of example.com/link/?link=...

#### Relevant Code:

```
const link: string = await admin.auth().generateEmailVerificationLink(someEmail, {url: '...', dynamicLinkDomain: '...'});
const link: string = await admin.auth().generatePasswordResetLink(someEmail, {url: '...', dynamicLinkDomain: '...'});
```

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez dans firebase-admin/lib/auth et examinez generatePasswordResetLink et generateEmailVerificationLink, ainsi que la gestion de ActionCodeSettings. Reproduisez l’exemple signalé avec un dynamicLinkDomain et un suffixe de lien personnalisé, puis vérifiez que les deux URL générées conservent le suffixe configuré avant de considérer l’issue comme terminée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
node.js, typescript
Domaine
authentication
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.