Daniel Bankhead
Daniel Bankhead
Typically, we do not want to emit declarations for code that has an `@internal` annotation. Documentation: - https://www.typescriptlang.org/tsconfig#stripInternal
It would be cool if this library included a simple, ready-to-go synchronous logger that: - Offers simplicity like `console.*` (one function call that prepares the `Entry` for customers) - Sends...
With the advent of TPC and PSC, there’s a lot of complexity with apiEndpoints. To improve, we should: - expose an `async Storage#getEndpoint()` method - make the `apiEndpoint` property private...
Today, `baseUrl` is equivalent to the `STORAGE_EMULATOR_HOST`, when available. This leads to [limitations](https://github.com/googleapis/nodejs-storage/issues/1314) when using the client. Here's the relevant code snippet: https://github.com/googleapis/nodejs-storage/blob/14472691121aa6af0a316c0bf78728edb164c66c/src/storage.ts#L650-L668 Ideally, `STORAGE_EMULATOR_HOST` should only be a host...
We should centralize shared headers. This can be made cleaner post-gaxios migration. Discussion: https://github.com/googleapis/nodejs-storage/pull/2305#discussion_r1336449564 This can also be iterated on in Node 18+ by returning a [Headers](https://developer.mozilla.org/en-US/docs/Web/API/Headers) object (as today...
**Is your feature request related to a problem? Please describe.** Required protobufs should be exported directly if required by the library. Currently, `google.protobuf` must be imported in order to be...
A proof-of-concept for migrating to native `fetch`. However, there are a few limitations today. ## No native `proxy` support It is possible to enable proxy support, however customers would have...
Fixes #429 Fixes #508 🦕
Remove `json-bigint` and `@types/json-bigint`. We can use native `bigint`s via the JSON.parse [reviver](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/JSON/parse) parameter and essentially check via: ```js // JSON spec for number: https://www.json.org/json-en.html const isInt = /^-?\d+(e(\+|\-)?\d+)?$/i.test(n); const...