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

Allow port numbers in authdomain #7277

Merged
merged 2 commits into from
May 5, 2023
Merged

Allow port numbers in authdomain #7277

merged 2 commits into from
May 5, 2023

Conversation

bhparijat
Copy link
Contributor

@bhparijat bhparijat commented May 4, 2023

Testing

Used demo app to test changes. The authdomain was set to localhost:8080 and I saw the redirect hitting URL
https://localhost:8080/__/auth/handler?apiKey=<API_KEY>&appName=%5BDEFAULT%5D&authType=signInViaRedirect&redirectUrl=http%3A%2F%2Flocalhost%3A5000%2F&v=9.8.1&providerId=google.com&scopes=profile

Fixes: #7233

@bhparijat bhparijat requested review from lisajian, prameshj, renkelvin, sam-gc and a team as code owners May 4, 2023 20:50
@changeset-bot
Copy link

changeset-bot bot commented May 4, 2023

🦋 Changeset detected

Latest commit: d8b4450

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@firebase/auth Patch
@firebase/auth-compat Patch
firebase Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@bhparijat bhparijat force-pushed the fix-port-authomain branch from b59c585 to e0c00a9 Compare May 4, 2023 21:05
@bhparijat bhparijat requested review from a team as code owners May 4, 2023 21:05
@google-oss-bot
Copy link
Contributor

google-oss-bot commented May 4, 2023

Size Report 1

Affected Products

  • @firebase/auth

    TypeBase (92e2279)Merge (68ce67d)Diff
    browser171 kB170 kB-119 B (-0.1%)
    cordova198 kB198 kB-119 B (-0.1%)
    esm5223 kB223 kB-119 B (-0.1%)
    main168 kB167 kB-119 B (-0.1%)
    module171 kB170 kB-119 B (-0.1%)
    react-native183 kB183 kB-119 B (-0.1%)
  • @firebase/auth/cordova

    TypeBase (92e2279)Merge (68ce67d)Diff
    browser198 kB198 kB-119 B (-0.1%)
    module198 kB198 kB-119 B (-0.1%)
  • @firebase/auth/internal

    TypeBase (92e2279)Merge (68ce67d)Diff
    browser181 kB181 kB-119 B (-0.1%)
    esm5237 kB237 kB-119 B (-0.1%)
    main204 kB204 kB-121 B (-0.1%)
    module181 kB181 kB-119 B (-0.1%)
  • @firebase/auth/react-native

    TypeBase (92e2279)Merge (68ce67d)Diff
    browser183 kB183 kB-119 B (-0.1%)
    module183 kB183 kB-119 B (-0.1%)
  • bundle

    TypeBase (92e2279)Merge (68ce67d)Diff
    auth (Anonymous)71.2 kB71.1 kB-85 B (-0.1%)
    auth (EmailAndPassword)76.2 kB76.1 kB-85 B (-0.1%)
    auth (GoogleFBTwitterGitHubPopup)97.4 kB97.3 kB-85 B (-0.1%)
    auth (GooglePopup)94.7 kB94.6 kB-85 B (-0.1%)
    auth (GoogleRedirect)94.9 kB94.8 kB-85 B (-0.1%)
    auth (Phone)81.1 kB81.0 kB-85 B (-0.1%)
  • firebase

    TypeBase (92e2279)Merge (68ce67d)Diff
    firebase-auth-compat.js132 kB132 kB-78 B (-0.1%)
    firebase-auth-cordova.js147 kB147 kB-72 B (-0.0%)
    firebase-auth-react-native.js160 kB160 kB-72 B (-0.0%)
    firebase-auth.js127 kB127 kB-71 B (-0.1%)
    firebase-compat.js773 kB773 kB-79 B (-0.0%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/8AghSYAFkw.html

@google-oss-bot
Copy link
Contributor

google-oss-bot commented May 4, 2023

Size Analysis Report 1

Affected Products

  • @firebase/auth

    • ActionCodeOperation

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.4 kB35.3 kB-84 B (-0.2%)
      size-with-ext-deps56.2 kB56.1 kB-85 B (-0.2%)
    • ActionCodeURL

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.2 kB36.1 kB-84 B (-0.2%)
      size-with-ext-deps57.2 kB57.2 kB-85 B (-0.1%)
    • AuthCredential

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.4 kB35.3 kB-84 B (-0.2%)
      size-with-ext-deps56.2 kB56.1 kB-85 B (-0.2%)
    • AuthErrorCodes

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.2 kB40.1 kB-84 B (-0.2%)
      size-with-ext-deps61.0 kB60.9 kB-85 B (-0.1%)
    • EmailAuthCredential

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size38.0 kB37.9 kB-84 B (-0.2%)
      size-with-ext-deps58.8 kB58.7 kB-85 B (-0.1%)
    • EmailAuthProvider

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size39.5 kB39.4 kB-84 B (-0.2%)
      size-with-ext-deps60.5 kB60.4 kB-85 B (-0.1%)
    • FacebookAuthProvider

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.2 kB40.1 kB-84 B (-0.2%)
      size-with-ext-deps61.0 kB60.9 kB-85 B (-0.1%)
    • FactorId

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps55.9 kB55.9 kB-85 B (-0.2%)
    • GithubAuthProvider

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.2 kB40.1 kB-84 B (-0.2%)
      size-with-ext-deps61.0 kB60.9 kB-85 B (-0.1%)
    • GoogleAuthProvider

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.2 kB40.1 kB-84 B (-0.2%)
      size-with-ext-deps61.0 kB60.9 kB-85 B (-0.1%)
    • OAuthCredential

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size37.3 kB37.2 kB-84 B (-0.2%)
      size-with-ext-deps58.1 kB58.0 kB-85 B (-0.1%)
    • OAuthProvider

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size41.1 kB41.0 kB-84 B (-0.2%)
      size-with-ext-deps61.9 kB61.8 kB-85 B (-0.1%)
    • OperationType

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • PhoneAuthCredential

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size37.3 kB37.2 kB-84 B (-0.2%)
      size-with-ext-deps58.1 kB58.0 kB-85 B (-0.1%)
    • PhoneAuthProvider

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size39.0 kB38.9 kB-84 B (-0.2%)
      size-with-ext-deps59.8 kB59.7 kB-85 B (-0.1%)
    • PhoneMultiFactorGenerator

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.0 kB35.9 kB-84 B (-0.2%)
      size-with-ext-deps56.8 kB56.7 kB-85 B (-0.1%)
    • ProviderId

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.3 kB35.2 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB56.0 kB-85 B (-0.2%)
    • RecaptchaVerifier

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size41.4 kB41.3 kB-84 B (-0.2%)
      size-with-ext-deps62.1 kB62.1 kB-85 B (-0.1%)
    • SAMLAuthProvider

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size41.5 kB41.4 kB-84 B (-0.2%)
      size-with-ext-deps62.2 kB62.2 kB-85 B (-0.1%)
    • SignInMethod

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.3 kB35.2 kB-84 B (-0.2%)
      size-with-ext-deps56.1 kB56.0 kB-85 B (-0.2%)
    • TotpMultiFactorGenerator

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size37.6 kB37.5 kB-84 B (-0.2%)
      size-with-ext-deps58.4 kB58.3 kB-85 B (-0.1%)
    • TotpSecret

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.1 kB36.0 kB-84 B (-0.2%)
      size-with-ext-deps56.9 kB56.8 kB-85 B (-0.1%)
    • TwitterAuthProvider

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.2 kB40.1 kB-84 B (-0.2%)
      size-with-ext-deps61.0 kB60.9 kB-85 B (-0.1%)
    • applyActionCode

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.3 kB35.2 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • beforeAuthStateChanged

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • browserLocalPersistence

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size38.6 kB38.5 kB-84 B (-0.2%)
      size-with-ext-deps59.4 kB59.3 kB-85 B (-0.1%)
    • browserPopupRedirectResolver

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size59.2 kB59.1 kB-84 B (-0.1%)
      size-with-ext-deps80.2 kB80.1 kB-85 B (-0.1%)
    • browserSessionPersistence

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size38.6 kB38.5 kB-84 B (-0.2%)
      size-with-ext-deps59.4 kB59.3 kB-85 B (-0.1%)
    • checkActionCode

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.4 kB36.3 kB-84 B (-0.2%)
      size-with-ext-deps57.2 kB57.1 kB-85 B (-0.1%)
    • confirmPasswordReset

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.3 kB35.2 kB-84 B (-0.2%)
      size-with-ext-deps56.1 kB56.0 kB-85 B (-0.2%)
    • connectAuthEmulator

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.9 kB36.8 kB-84 B (-0.2%)
      size-with-ext-deps57.7 kB57.6 kB-85 B (-0.1%)
    • createUserWithEmailAndPassword

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size37.0 kB36.9 kB-84 B (-0.2%)
      size-with-ext-deps57.8 kB57.7 kB-85 B (-0.1%)
    • debugErrorMap

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size47.1 kB47.0 kB-84 B (-0.2%)
      size-with-ext-deps67.9 kB67.8 kB-85 B (-0.1%)
    • deleteUser

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps55.9 kB55.9 kB-85 B (-0.2%)
    • fetchSignInMethodsForEmail

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.5 kB35.4 kB-84 B (-0.2%)
      size-with-ext-deps56.3 kB56.2 kB-85 B (-0.2%)
    • getAdditionalUserInfo

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.5 kB36.4 kB-84 B (-0.2%)
      size-with-ext-deps57.3 kB57.2 kB-85 B (-0.1%)
    • getAuth

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size69.3 kB69.2 kB-84 B (-0.1%)
      size-with-ext-deps96.7 kB96.7 kB-85 B (-0.1%)
    • getIdToken

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • getIdTokenResult

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.1 kB35.0 kB-84 B (-0.2%)
      size-with-ext-deps55.9 kB55.8 kB-85 B (-0.2%)
    • getMultiFactorResolver

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size38.0 kB38.0 kB-84 B (-0.2%)
      size-with-ext-deps58.8 kB58.7 kB-85 B (-0.1%)
    • getRedirectResult

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size42.7 kB42.6 kB-84 B (-0.2%)
      size-with-ext-deps63.5 kB63.4 kB-85 B (-0.1%)
    • inMemoryPersistence

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.1 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps55.9 kB55.8 kB-85 B (-0.2%)
    • indexedDBLocalPersistence

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size42.6 kB42.5 kB-84 B (-0.2%)
      size-with-ext-deps63.3 kB63.3 kB-85 B (-0.1%)
    • initializeAuth

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.4 kB35.3 kB-84 B (-0.2%)
      size-with-ext-deps56.7 kB56.6 kB-85 B (-0.2%)
    • initializeRecaptchaConfig

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • isSignInWithEmailLink

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.3 kB36.3 kB-84 B (-0.2%)
      size-with-ext-deps57.4 kB57.3 kB-85 B (-0.1%)
    • linkWithCredential

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.2 kB36.1 kB-84 B (-0.2%)
      size-with-ext-deps57.0 kB56.9 kB-85 B (-0.1%)
    • linkWithPhoneNumber

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size39.7 kB39.6 kB-84 B (-0.2%)
      size-with-ext-deps60.5 kB60.4 kB-85 B (-0.1%)
    • linkWithPopup

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size46.3 kB46.2 kB-84 B (-0.2%)
      size-with-ext-deps67.1 kB67.0 kB-85 B (-0.1%)
    • linkWithRedirect

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size41.5 kB41.4 kB-84 B (-0.2%)
      size-with-ext-deps62.3 kB62.2 kB-85 B (-0.1%)
    • multiFactor

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size37.4 kB37.3 kB-84 B (-0.2%)
      size-with-ext-deps58.1 kB58.1 kB-85 B (-0.1%)
    • onAuthStateChanged

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • onIdTokenChanged

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • parseActionCodeURL

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.3 kB36.2 kB-84 B (-0.2%)
      size-with-ext-deps57.3 kB57.2 kB-85 B (-0.1%)
    • prodErrorMap

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.1 kB35.0 kB-84 B (-0.2%)
      size-with-ext-deps55.9 kB55.8 kB-85 B (-0.2%)
    • reauthenticateWithCredential

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.7 kB36.7 kB-84 B (-0.2%)
      size-with-ext-deps57.5 kB57.4 kB-85 B (-0.1%)
    • reauthenticateWithPhoneNumber

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.3 kB40.2 kB-84 B (-0.2%)
      size-with-ext-deps61.1 kB61.0 kB-85 B (-0.1%)
    • reauthenticateWithPopup

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size46.3 kB46.2 kB-84 B (-0.2%)
      size-with-ext-deps67.1 kB67.0 kB-85 B (-0.1%)
    • reauthenticateWithRedirect

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size41.2 kB41.2 kB-84 B (-0.2%)
      size-with-ext-deps62.0 kB61.9 kB-85 B (-0.1%)
    • reload

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.1 kB35.0 kB-84 B (-0.2%)
      size-with-ext-deps55.9 kB55.8 kB-85 B (-0.2%)
    • sendEmailVerification

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.1 kB36.0 kB-84 B (-0.2%)
      size-with-ext-deps56.9 kB56.8 kB-85 B (-0.1%)
    • sendPasswordResetEmail

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.8 kB36.8 kB-84 B (-0.2%)
      size-with-ext-deps57.6 kB57.5 kB-85 B (-0.1%)
    • sendSignInLinkToEmail

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.9 kB36.8 kB-84 B (-0.2%)
      size-with-ext-deps57.7 kB57.6 kB-85 B (-0.1%)
    • setPersistence

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • signInAnonymously

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.3 kB36.2 kB-84 B (-0.2%)
      size-with-ext-deps57.1 kB57.0 kB-85 B (-0.1%)
    • signInWithCredential

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.5 kB36.4 kB-84 B (-0.2%)
      size-with-ext-deps57.3 kB57.2 kB-85 B (-0.1%)
    • signInWithCustomToken

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.2 kB36.1 kB-84 B (-0.2%)
      size-with-ext-deps56.9 kB56.9 kB-85 B (-0.1%)
    • signInWithEmailAndPassword

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.9 kB40.8 kB-84 B (-0.2%)
      size-with-ext-deps62.0 kB61.9 kB-85 B (-0.1%)
    • signInWithEmailLink

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size41.1 kB41.0 kB-84 B (-0.2%)
      size-with-ext-deps62.2 kB62.1 kB-85 B (-0.1%)
    • signInWithPhoneNumber

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.1 kB40.0 kB-84 B (-0.2%)
      size-with-ext-deps60.9 kB60.8 kB-85 B (-0.1%)
    • signInWithPopup

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size46.2 kB46.2 kB-84 B (-0.2%)
      size-with-ext-deps67.0 kB67.0 kB-85 B (-0.1%)
    • signInWithRedirect

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size40.9 kB40.9 kB-84 B (-0.2%)
      size-with-ext-deps61.7 kB61.6 kB-85 B (-0.1%)
    • signOut

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps55.9 kB55.9 kB-85 B (-0.2%)
    • unlink

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.7 kB35.7 kB-84 B (-0.2%)
      size-with-ext-deps56.5 kB56.4 kB-85 B (-0.2%)
    • updateCurrentUser

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • updateEmail

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.5 kB35.4 kB-84 B (-0.2%)
      size-with-ext-deps56.2 kB56.1 kB-85 B (-0.2%)
    • updatePassword

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.5 kB35.4 kB-84 B (-0.2%)
      size-with-ext-deps56.2 kB56.1 kB-85 B (-0.2%)
    • updatePhoneNumber

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.9 kB35.8 kB-84 B (-0.2%)
      size-with-ext-deps56.7 kB56.6 kB-85 B (-0.1%)
    • updateProfile

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.6 kB35.6 kB-84 B (-0.2%)
      size-with-ext-deps56.4 kB56.3 kB-85 B (-0.2%)
    • useDeviceLanguage

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size35.2 kB35.1 kB-84 B (-0.2%)
      size-with-ext-deps56.0 kB55.9 kB-85 B (-0.2%)
    • verifyBeforeUpdateEmail

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.1 kB36.0 kB-84 B (-0.2%)
      size-with-ext-deps56.9 kB56.8 kB-85 B (-0.1%)
    • verifyPasswordResetCode

      Size

      TypeBase (92e2279)Merge (ad4c44d)Diff
      size36.4 kB36.4 kB-84 B (-0.2%)
      size-with-ext-deps57.2 kB57.1 kB-85 B (-0.1%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/zGQiB7Ndbk.html

@renkelvin
Copy link

Test Modified Packages / Test Packages With Changed Files in Firefox (pull_request) seems timeout. Please try run it again.
Otherwise LGTM.

@bhparijat bhparijat force-pushed the fix-port-authomain branch from e53f841 to 43c8261 Compare May 5, 2023 03:39
@bhparijat bhparijat force-pushed the fix-port-authomain branch from 97fc169 to 03e6314 Compare May 5, 2023 20:15
@bhparijat bhparijat force-pushed the fix-port-authomain branch from 03e6314 to d8b4450 Compare May 5, 2023 20:16
@bhparijat bhparijat requested a review from prameshj May 5, 2023 20:23
Copy link
Contributor

@prameshj prameshj left a comment

Choose a reason for hiding this comment

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

Thanks!

@bhparijat bhparijat merged commit afdccd5 into master May 5, 2023
@bhparijat bhparijat deleted the fix-port-authomain branch May 5, 2023 21:40
@google-oss-bot google-oss-bot mentioned this pull request May 11, 2023
@firebase firebase locked and limited conversation to collaborators Jun 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Getting FirebaseError (auth/argument-error) if authDomain URL specifies port number
6 participants