firebase / firebase/firebase-admin-node

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

Offen
#1,556 4 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
api: auth
Vorherrschende Sprache
TypeScript
Sterne
1.7k
Forks
419
Ø Merge
3 T. 10 Std.
Gemergte PRs (30 T.)
16

Beschreibung

* 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: '...'});
```

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in firebase-admin/lib/auth und untersuche generatePasswordResetLink und generateEmailVerificationLink zusammen mit der Verarbeitung von ActionCodeSettings. Reproduziere das gemeldete Beispiel mit einer dynamicLinkDomain und einem benutzerdefinierten Link-Suffix und überprüfe anschließend, dass beide generierten URLs das konfigurierte Suffix beibehalten, bevor du das Issue als erledigt betrachtest.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
node.js, typescript
Bereich
authentication
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.