Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(auth): Add TOTP support in Project and Tenant config #1989

Merged
merged 57 commits into from Mar 29, 2023

Conversation

pragatimodi
Copy link
Contributor

@pragatimodi pragatimodi commented Nov 23, 2022

DO_NOT_MERGE (waiting for BE changes)

PR for TOTP Config for Project and Tenant

RELEASE NOTES: Added support for TOTP (Time-based One Time Password) as a Multi-Factor Authentication option in Tenant and Project config.

pragatimodi and others added 5 commits November 22, 2022 18:42
* build(deps): bump @types/node from 18.7.23 to 18.11.9 (#1983)

Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 18.7.23 to 18.11.9.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps-dev): bump @firebase/auth-types from 0.11.0 to 0.11.1 (#1985)

* build(deps-dev): bump sinon from 14.0.1 to 14.0.2 (#1984)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* build(deps): bump @types/node from 18.7.23 to 18.11.9 (#1983)

Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 18.7.23 to 18.11.9.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps-dev): bump @firebase/auth-types from 0.11.0 to 0.11.1 (#1985)

* build(deps-dev): bump sinon from 14.0.1 to 14.0.2 (#1984)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@pragatimodi pragatimodi marked this pull request as draft November 23, 2022 03:02
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Show resolved Hide resolved
src/auth/project-config.ts Outdated Show resolved Hide resolved
@prameshj
Copy link

prameshj commented Dec 1, 2022

Also adding @Xiaoshouzi-gh to take a look.

@Xiaoshouzi-gh
Copy link
Contributor

@pragatimodi for failed check, you can try $ npm run api-extractor:local and commit all the auto-generated files.

src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Show resolved Hide resolved
test/unit/auth/auth-config.spec.ts Show resolved Hide resolved
src/auth/project-config.ts Outdated Show resolved Hide resolved
src/auth/project-config.ts Outdated Show resolved Hide resolved
etc/firebase-admin.auth.api.md Outdated Show resolved Hide resolved
test/integration/auth.spec.ts Outdated Show resolved Hide resolved
test/integration/auth.spec.ts Outdated Show resolved Hide resolved
test/unit/auth/auth-config.spec.ts Outdated Show resolved Hide resolved
test/unit/auth/auth-config.spec.ts Outdated Show resolved Hide resolved
test/unit/auth/tenant.spec.ts Outdated Show resolved Hide resolved
test/integration/auth.spec.ts Show resolved Hide resolved
test/integration/auth.spec.ts Outdated Show resolved Hide resolved
etc/firebase-admin.auth.api.md Outdated Show resolved Hide resolved
package-lock.json Outdated Show resolved Hide resolved
src/auth/auth-api-request.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/project-config.ts Outdated Show resolved Hide resolved
test/integration/auth.spec.ts Outdated Show resolved Hide resolved
src/auth/project-config.ts Outdated Show resolved Hide resolved
src/auth/project-config.ts Outdated Show resolved Hide resolved
test/unit/auth/auth-config.spec.ts Outdated Show resolved Hide resolved
src/auth/project-config.ts Outdated Show resolved Hide resolved
etc/firebase-admin.auth.api.md Outdated Show resolved Hide resolved
@Xiaoshouzi-gh
Copy link
Contributor

LGTM, waiting on the test cases explanation docs (internal).

src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/project-config.ts Show resolved Hide resolved
Copy link
Member

@lahirumaramba lahirumaramba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you!

Please get the docs reviewed before you merge this. @kevinthecheung might be able to help. Thanks!

src/auth/auth-config.ts Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/auth-config.ts Outdated Show resolved Hide resolved
src/auth/project-config.ts Outdated Show resolved Hide resolved
@pragatimodi pragatimodi merged commit 0ef5cfc into master Mar 29, 2023
5 checks passed
@lahirumaramba lahirumaramba deleted the totp-release branch April 5, 2023 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:stage Stage a release candidate release-note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants