firebase / firebase/firebase-admin-node
[firebase-admin] dynamic custom link suffix not added in email verification links in auth admin
- 主要言語
- TypeScript
- スター
- 1.7k
- フォーク
- 419
- 平均マージ
- 3日 10時間
- マージ済み PR(30日)
- 16
説明
* 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: '...'});
```
コントリビューションガイド
調査の方向性
firebase-admin/lib/auth から始めて、generatePasswordResetLink と generateEmailVerificationLink、および ActionCodeSettings の処理を確認します。dynamicLinkDomain とカスタムリンクサフィックスを使って報告された例を再現し、その後、生成された両方の URL に設定したサフィックスが保持されていることを確認してから、issue の対応完了とみなします。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- node.js, typescript
- 領域
- authentication
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100