Skip to content

Change download link to not require storage.getMetadata() ? #524

Closed
Closed
Change download link to not require storage.getMetadata() ?#524
Assignees
stephenplusplus
Labels
api: storageIssues related to the Cloud Storage API.
Milestone
Storage Stable

Description

@jgeewax
jgeewax
opened on May 4, 2015

Looks like we can auto-generate the download link without pulling all the metadata in a separate request:

https://<bucket>.storage.googleapis.com/<object>

Can we just make this a standard URL?

@robertdimarco says:

Remove the getMetadata() round-trip from the createReadStream() code path, in the interests of reducing overall request latency.

In comparison to the S3 implementation we were using, gcloud-node performs an additional call to the storage backend in series to get the metadata to obtain a download link. This download link, however, is able to be statically generated. Removing the extra round-trip to the server would halve the request latency for serving small files.

Activity

  1. jgeewax
    added
    enhancement
    perf
    api: storageIssues related to the Cloud Storage API.
    on May 4, 2015
  2. jgeewax
    added this to the Storage Stable milestone on May 4, 2015
  3. stephenplusplus
    self-assigned this
    on May 4, 2015
  4. stephenplusplus
    added 2 commits that reference this issue on May 4, 2015

    storage: use standard URL instead of calling getMetadata - fixes goog…

    f79550b

    storage: use standard URL instead of calling getMetadata - fixes goog…

    93d7a12
  5. stephenplusplus
    closed this as completedin 93d7a12on May 4, 2015
  6. ryanseys
    added a commit that references this issue on May 4, 2015

    Merge pull request #525 from stephenplusplus/spp--storage-change-down…

    98a2153
  7. sofisl
    added a commit that references this issue on Sep 15, 2022

    chore(deps): update dependency sinon to v12 (#524)

    d7c8a6d
  8. sofisl
    added 2 commits that reference this issue on Oct 11, 2022

    docs: updated code of conduct (includes update to actions) (#524)

    ce2776e

    docs: updated code of conduct (includes update to actions) (#524)

    9d17fb2
  9. sofisl
    added 3 commits that reference this issue on Nov 11, 2022

    fix(deps): google-gax v2.24.1 (#524)

    a633c4d

    chore: release 2.4.4 (#522)

    b374899

    chore: update v2.14.2 gapic-generator-typescript (#524)

    e3aa36b

  10. 4 remaining items

  11. sofisl
    added 3 commits that reference this issue on Sep 13, 2023

    fix(deps): require google-gax v2.17.0 (#524)

    ad278bb

    chore: release 2.4.3 (#520)

    b341a03

    build!: update library to use Node 12 (#524)

    7341476
  12. miguelvelezsa
    added a commit that references this issue on Jul 23, 2025

    chore: release 1.1.3 (#524)

    f137324
  13. GautamSharda
    added 2 commits that reference this issue on Jan 14, 2026

    chore(deps): update dependency gaxios to v4 (#524)

    87d7294

    chore(deps): update dependency gaxios to v4 (#524)

    9ce6e9b
  14. miguelvelezsa
    added a commit that references this issue on Jan 28, 2026

    fix(types): removed duplicate declaration (#524)

    c06624e
  15. sofisl
    added 2 commits that reference this issue on Feb 3, 2026

    chore(deps): update dependency typescript to ~3.6.0 (#524)

    a374f8a

    fix(types): removed duplicate declaration (#524)

    bf69c0e
  16. GautamSharda
    added a commit that references this issue on Feb 5, 2026

    chore(deps): update dependency gaxios to v4 (#524)

    a71e8c8
  17. sofisl
    added a commit that references this issue on Feb 17, 2026

    fix(types): removed duplicate declaration (#524)

    f908810
  18. sofisl
    added 2 commits that reference this issue on Feb 24, 2026

    docs: fix import links in the jsdocs (#524)

    427d9f5

    fix(deps): update dependency @google-cloud/promisify to ^0.4.0 (#524)

    949cbf7
  19. GautamSharda
    added a commit that references this issue on Mar 5, 2026

    build: use config file for linkinator (#524)

    6ec6bf2
  20. sofisl
    added a commit that references this issue on Mar 5, 2026

    fix(deps): update dependency protobufjs to v7.5.4 (#524)

    17bc8cf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

api: storageIssues related to the Cloud Storage API.

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions