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

Add Env Override Global Variable forceEnvironment #6901

Merged
merged 9 commits into from
Dec 22, 2022

Conversation

dwyfrequency
Copy link
Contributor

To adapt to the changing javascript landscape, we will allow users to specify their environment as Node or Browser to override Firebase's runtime environment detection and force the SDK to act as if it were in the respective environment.

@changeset-bot
Copy link

changeset-bot bot commented Dec 19, 2022

🦋 Changeset detected

Latest commit: 409f9c5

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

This PR includes changesets to release 28 packages
Name Type
@firebase/util Minor
firebase Minor
@firebase/analytics-compat Patch
@firebase/analytics Patch
@firebase/app-check-compat Patch
@firebase/app-check Patch
@firebase/app-compat Patch
@firebase/app Patch
@firebase/auth-compat Patch
@firebase/auth Patch
@firebase/component Patch
@firebase/database-compat Patch
@firebase/database-types Patch
@firebase/database Patch
@firebase/firestore-compat Patch
@firebase/firestore Patch
@firebase/functions-compat Patch
@firebase/functions Patch
@firebase/installations-compat Patch
@firebase/installations Patch
@firebase/messaging-compat Patch
@firebase/messaging Patch
@firebase/performance-compat Patch
@firebase/performance Patch
@firebase/remote-config-compat Patch
@firebase/remote-config Patch
@firebase/storage-compat Patch
@firebase/storage 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

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Dec 19, 2022

Size Report 1

Affected Products

  • @firebase/firestore

    TypeBase (d8af08f)Merge (35a3764)Diff
    browser272 kB273 kB+52 B (+0.0%)
    esm5339 kB339 kB+69 B (+0.0%)
    main545 kB545 kB+120 B (+0.0%)
    module272 kB273 kB+52 B (+0.0%)
    react-native273 kB273 kB+52 B (+0.0%)
  • @firebase/util

    TypeBase (d8af08f)Merge (35a3764)Diff
    browser22.7 kB22.9 kB+201 B (+0.9%)
    esm524.2 kB24.4 kB+199 B (+0.8%)
    main29.7 kB30.0 kB+273 B (+0.9%)
    module22.7 kB22.9 kB+201 B (+0.9%)
  • bundle

    TypeBase (d8af08f)Merge (35a3764)Diff
    firestore (Persistence)281 kB281 kB+182 B (+0.1%)
    firestore (Query Cursors)220 kB220 kB+52 B (+0.0%)
    firestore (Query)217 kB217 kB+52 B (+0.0%)
    firestore (Read data once)205 kB205 kB+52 B (+0.0%)
    firestore (Realtime updates)207 kB207 kB+52 B (+0.0%)
    firestore (Transaction)188 kB188 kB+52 B (+0.0%)
    firestore (Write data)188 kB188 kB+52 B (+0.0%)
    storage (uploadBytes)46.8 kB46.9 kB+130 B (+0.3%)
    storage (uploadBytesResumable)56.8 kB56.9 kB+130 B (+0.2%)
    storage (uploadString)47.0 kB47.2 kB+130 B (+0.3%)
  • firebase

    TypeBase (d8af08f)Merge (35a3764)Diff
    firebase-app.js92.0 kB92.8 kB+831 B (+0.9%)
    firebase-auth-compat.js126 kB126 kB+113 B (+0.1%)
    firebase-auth-react-native.js150 kB150 kB+130 B (+0.1%)
    firebase-compat.js745 kB746 kB+157 B (+0.0%)
    firebase-firestore-compat.js320 kB320 kB+917 B (+0.3%)
    firebase-firestore.js321 kB321 kB+168 B (+0.1%)
    firebase-performance-standalone-compat.es2017.js89.9 kB89.9 kB+1 B (+0.0%)
    firebase-storage-compat.js39.4 kB40.3 kB+878 B (+2.2%)
    firebase-storage.js39.0 kB39.1 kB+116 B (+0.3%)

Test Logs

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

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Dec 19, 2022

Size Analysis Report 1

Affected Products

  • @firebase/firestore

    • addDoc

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size118 kB118 kB+52 B (+0.0%)
      size-with-ext-deps178 kB178 kB+52 B (+0.0%)
    • deleteDoc

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size109 kB109 kB+52 B (+0.0%)
      size-with-ext-deps169 kB169 kB+52 B (+0.0%)
    • disableNetwork

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size97.5 kB97.6 kB+52 B (+0.1%)
      size-with-ext-deps157 kB157 kB+52 B (+0.0%)
    • enableIndexedDbPersistence

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size179 kB179 kB+52 B (+0.0%)
      size-with-ext-deps239 kB240 kB+1.01 kB (+0.4%)
    • enableMultiTabIndexedDbPersistence

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size211 kB211 kB+52 B (+0.0%)
      size-with-ext-deps271 kB272 kB+1.01 kB (+0.4%)
    • enableNetwork

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size97.5 kB97.6 kB+52 B (+0.1%)
      size-with-ext-deps157 kB157 kB+52 B (+0.0%)
    • executeWrite

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size108 kB108 kB+52 B (+0.0%)
      size-with-ext-deps168 kB168 kB+52 B (+0.0%)
    • getCountFromServer

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size104 kB104 kB+52 B (+0.0%)
      size-with-ext-deps164 kB164 kB+52 B (+0.0%)
    • getDoc

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size135 kB135 kB+52 B (+0.0%)
      size-with-ext-deps194 kB194 kB+52 B (+0.0%)
    • getDocFromCache

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size90.6 kB90.7 kB+52 B (+0.1%)
      size-with-ext-deps150 kB150 kB+52 B (+0.0%)
    • getDocFromServer

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size135 kB135 kB+52 B (+0.0%)
      size-with-ext-deps194 kB194 kB+52 B (+0.0%)
    • getDocs

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size136 kB136 kB+52 B (+0.0%)
      size-with-ext-deps196 kB196 kB+52 B (+0.0%)
    • getDocsFromCache

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size98.3 kB98.3 kB+52 B (+0.1%)
      size-with-ext-deps157 kB157 kB+52 B (+0.0%)
    • getDocsFromServer

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size136 kB136 kB+52 B (+0.0%)
      size-with-ext-deps196 kB196 kB+52 B (+0.0%)
    • loadBundle

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size107 kB107 kB+52 B (+0.0%)
      size-with-ext-deps166 kB166 kB+52 B (+0.0%)
    • namedQuery

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size84.1 kB84.1 kB+52 B (+0.1%)
      size-with-ext-deps143 kB143 kB+52 B (+0.0%)
    • onSnapshot

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size137 kB137 kB+52 B (+0.0%)
      size-with-ext-deps196 kB197 kB+52 B (+0.0%)
    • onSnapshotsInSync

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size127 kB127 kB+52 B (+0.0%)
      size-with-ext-deps186 kB186 kB+52 B (+0.0%)
    • runTransaction

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size118 kB118 kB+52 B (+0.0%)
      size-with-ext-deps178 kB178 kB+52 B (+0.0%)
    • setDoc

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size117 kB117 kB+52 B (+0.0%)
      size-with-ext-deps177 kB177 kB+52 B (+0.0%)
    • setIndexConfiguration

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size86.7 kB86.7 kB+52 B (+0.1%)
      size-with-ext-deps146 kB146 kB+52 B (+0.0%)
    • updateDoc

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size118 kB118 kB+52 B (+0.0%)
      size-with-ext-deps177 kB177 kB+52 B (+0.0%)
    • waitForPendingWrites

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size97.9 kB98.0 kB+52 B (+0.1%)
      size-with-ext-deps157 kB157 kB+52 B (+0.0%)
    • writeBatch

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size120 kB120 kB+52 B (+0.0%)
      size-with-ext-deps179 kB179 kB+52 B (+0.0%)
  • @firebase/storage

    • _FbsBlob

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size-with-ext-deps31.4 kB32.4 kB+955 B (+3.0%)
    • _UploadTask

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size-with-ext-deps47.0 kB47.9 kB+955 B (+2.0%)
    • uploadBytes

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size-with-ext-deps37.2 kB38.2 kB+961 B (+2.6%)
    • uploadBytesResumable

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size-with-ext-deps47.2 kB48.2 kB+961 B (+2.0%)
    • uploadString

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size-with-ext-deps37.5 kB38.4 kB+961 B (+2.6%)
  • @firebase/util

    • getDefaults

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size?3.83 kB? (?)
      size-with-ext-deps?3.83 kB? (?)

      Dependency

      TypeBase (d8af08f)Merge (35a3764)Diff
      functions?

      getGlobal

      ?
      classes??
      variables?

      base64
      base64Decode
      byteArrayToString
      getDefaults
      getDefaultsFromCookie
      getDefaultsFromEnvVariable
      getDefaultsFromGlobal
      stringToByteArray$1

      ?
      enums??
    • isNode

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size135 B4.07 kB+3.94 kB (+2917.8%)
      size-with-ext-deps135 B4.07 kB+3.94 kB (+2917.8%)

      Dependency

      TypeBase (d8af08f)Merge (35a3764)Diff
      functions

      isNode

      getGlobal
      isNode

      + getGlobal

      variables

      base64
      base64Decode
      byteArrayToString
      getDefaults
      getDefaultsFromCookie
      getDefaultsFromEnvVariable
      getDefaultsFromGlobal
      stringToByteArray$1

      + base64
      + base64Decode
      + byteArrayToString
      + getDefaults
      + getDefaultsFromCookie
      + getDefaultsFromEnvVariable
      + getDefaultsFromGlobal
      + stringToByteArray$1

    • isSafari

      Size

      TypeBase (d8af08f)Merge (35a3764)Diff
      size242 B4.18 kB+3.94 kB (+1627.7%)
      size-with-ext-deps242 B4.18 kB+3.94 kB (+1627.7%)

      Dependency

      TypeBase (d8af08f)Merge (35a3764)Diff
      functions

      isNode
      isSafari

      getGlobal
      isNode
      isSafari

      + getGlobal

      variables

      base64
      base64Decode
      byteArrayToString
      getDefaults
      getDefaultsFromCookie
      getDefaultsFromEnvVariable
      getDefaultsFromGlobal
      stringToByteArray$1

      + base64
      + base64Decode
      + byteArrayToString
      + getDefaults
      + getDefaultsFromCookie
      + getDefaultsFromEnvVariable
      + getDefaultsFromGlobal
      + stringToByteArray$1

Test Logs

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

@github-actions github-actions bot added the doc-changes PRs that affect docs label Dec 19, 2022
@dwyfrequency dwyfrequency marked this pull request as ready for review December 20, 2022 19:33
@github-actions
Copy link
Contributor

github-actions bot commented Dec 20, 2022

Changeset File Check ✅

  • No modified packages are missing from the changeset file.
  • No changeset formatting errors detected.

Copy link
Contributor

@hsubox76 hsubox76 left a comment

Choose a reason for hiding this comment

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

Looks good! Just a few nits about documentation and such. When we do this release we'll have to remember to generate new reference docs.

Copy link
Contributor

@hsubox76 hsubox76 left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for getting this done!

@dwyfrequency dwyfrequency merged commit 06dc136 into master Dec 22, 2022
@dwyfrequency dwyfrequency deleted the jd-env-default-override branch December 22, 2022 21:27
@google-oss-bot google-oss-bot mentioned this pull request Jan 17, 2023
@firebase firebase locked and limited conversation to collaborators Jan 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
doc-changes PRs that affect docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants