Skip to content

clear the global timeout once an operation is done in the Storage SDK #5703

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

Merged
merged 3 commits into from
Nov 10, 2021

Conversation

Feiyang1
Copy link
Member

@Feiyang1 Feiyang1 commented Nov 5, 2021

Fixes #5628

@changeset-bot
Copy link

changeset-bot bot commented Nov 5, 2021

🦋 Changeset detected

Latest commit: 1265a79

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

This PR includes changesets to release 3 packages
Name Type
@firebase/storage Patch
firebase Patch
@firebase/storage-compat 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-cla google-cla bot added the cla: yes label Nov 5, 2021
@Feiyang1 Feiyang1 requested a review from egilmorez as a code owner November 5, 2021 00:04
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Nov 5, 2021

Binary Size Report

Affected SDKs

  • @firebase/storage

    Type Base (dbd54f7) Head (45c4365) Diff
    browser 52.1 kB 52.4 kB +231 B (+0.4%)
    esm5 57.5 kB 57.8 kB +231 B (+0.4%)
    main 53.7 kB 54.0 kB +231 B (+0.4%)
    module 52.1 kB 52.4 kB +231 B (+0.4%)
  • firebase

    Type Base (dbd54f7) Head (45c4365) Diff
    firebase-compat.js 749 kB 749 kB +75 B (+0.0%)
    firebase-storage-compat.js 38.0 kB 38.1 kB +75 B (+0.2%)
    firebase-storage.js 139 kB 140 kB +411 B (+0.3%)

Test Logs

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Nov 5, 2021

Size Analysis Report

Affected Products

  • @firebase/storage

    • _FbsBlob

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      13.9 kB
      13.9 kB
      +65 B (+0.5%)
      size-with-ext-deps
      19.8 kB
      19.9 kB
      +65 B (+0.3%)
    • _Location

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      9.90 kB
      9.96 kB
      +65 B (+0.7%)
      size-with-ext-deps
      15.8 kB
      15.9 kB
      +65 B (+0.4%)
    • _UploadTask

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      28.7 kB
      28.8 kB
      +65 B (+0.2%)
      size-with-ext-deps
      34.7 kB
      34.7 kB
      +65 B (+0.2%)
    • _dataFromString

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      12.1 kB
      12.2 kB
      +65 B (+0.5%)
      size-with-ext-deps
      18.1 kB
      18.1 kB
      +65 B (+0.4%)
    • _getChild

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      10.2 kB
      10.2 kB
      +65 B (+0.6%)
      size-with-ext-deps
      16.1 kB
      16.1 kB
      +65 B (+0.4%)
    • _invalidArgument

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      9.91 kB
      9.97 kB
      +65 B (+0.7%)
      size-with-ext-deps
      15.8 kB
      15.9 kB
      +65 B (+0.4%)
    • _invalidRootOperation

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      9.91 kB
      9.98 kB
      +65 B (+0.7%)
      size-with-ext-deps
      15.8 kB
      15.9 kB
      +65 B (+0.4%)
    • connectStorageEmulator

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      10.1 kB
      10.2 kB
      +65 B (+0.6%)
      size-with-ext-deps
      19.7 kB
      19.8 kB
      +65 B (+0.3%)
    • deleteObject

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      13.0 kB
      13.1 kB
      +65 B (+0.5%)
      size-with-ext-deps
      19.0 kB
      19.1 kB
      +65 B (+0.3%)
    • getDownloadURL

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      15.2 kB
      15.3 kB
      +65 B (+0.4%)
      size-with-ext-deps
      21.2 kB
      21.3 kB
      +65 B (+0.3%)
    • getMetadata

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      14.6 kB
      14.7 kB
      +65 B (+0.4%)
      size-with-ext-deps
      20.6 kB
      20.7 kB
      +65 B (+0.3%)
    • getStorage

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      10.0 kB
      10.1 kB
      +65 B (+0.6%)
      size-with-ext-deps
      16.2 kB
      16.2 kB
      +65 B (+0.4%)
    • list

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      13.8 kB
      13.8 kB
      +65 B (+0.5%)
      size-with-ext-deps
      19.8 kB
      19.8 kB
      +65 B (+0.3%)
    • listAll

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      14.0 kB
      14.1 kB
      +65 B (+0.5%)
      size-with-ext-deps
      20.0 kB
      20.1 kB
      +65 B (+0.3%)
    • ref

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      10.8 kB
      10.8 kB
      +65 B (+0.6%)
      size-with-ext-deps
      16.7 kB
      16.8 kB
      +65 B (+0.4%)
    • updateMetadata

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      14.9 kB
      15.0 kB
      +65 B (+0.4%)
      size-with-ext-deps
      20.9 kB
      21.0 kB
      +65 B (+0.3%)
    • uploadBytes

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      19.4 kB
      19.5 kB
      +65 B (+0.3%)
      size-with-ext-deps
      25.5 kB
      25.5 kB
      +65 B (+0.3%)
    • uploadBytesResumable

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      28.9 kB
      28.9 kB
      +65 B (+0.2%)
      size-with-ext-deps
      34.9 kB
      34.9 kB
      +65 B (+0.2%)
    • uploadString

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      19.6 kB
      19.7 kB
      +65 B (+0.3%)
      size-with-ext-deps
      25.7 kB
      25.7 kB
      +65 B (+0.3%)
    • StringFormat

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      9.98 kB
      10.0 kB
      +65 B (+0.7%)
      size-with-ext-deps
      15.9 kB
      16.0 kB
      +65 B (+0.4%)
    • _TaskEvent

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      9.94 kB
      10.0 kB
      +65 B (+0.7%)
      size-with-ext-deps
      15.9 kB
      15.9 kB
      +65 B (+0.4%)
    • _TaskState

      Size Table

      TypeBase (dbd54f7)Head (209d417)Diff
      size
      10.0 kB
      10.1 kB
      +65 B (+0.7%)
      size-with-ext-deps
      15.9 kB
      16.0 kB
      +65 B (+0.4%)

@Feiyang1 Feiyang1 merged commit e0fe2b6 into master Nov 10, 2021
@Feiyang1 Feiyang1 deleted the fei-storage-hang branch November 10, 2021 18:00
@google-oss-bot google-oss-bot mentioned this pull request Nov 10, 2021
@firebase firebase locked and limited conversation to collaborators Dec 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Firebase Storage getDownloadURL doesn't finish running
3 participants