Skip to content

Commit

Permalink
docs: fix server timing docs and generate new readme
Browse files Browse the repository at this point in the history
  • Loading branch information
SgtPooki committed Jan 9, 2025
1 parent 1586f62 commit b410346
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
8 changes: 8 additions & 0 deletions packages/verified-fetch/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -635,6 +635,14 @@ Some known header specifications:
- <https://specs.ipfs.tech/http-gateways/trustless-gateway/#response-headers>
- <https://specs.ipfs.tech/http-gateways/subdomain-gateway/#response-headers>

#### Server Timing headers

By default, we do not include Server Timing headers in responses. If you want to include them, you can pass an
`includeServerTiming` option to the `createVerifiedFetch` function to include them in all future responses. You can
also pass the `includeServerTiming` option to each fetch call to include them only for that specific response.

See PR where this was added, <https://github.com/ipfs/helia-verified-fetch/pull/164>, for more information.

### Possible Scenarios that could cause confusion

#### Attempting to fetch the CID for content that does not make sense
Expand Down
4 changes: 2 additions & 2 deletions packages/verified-fetch/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -606,11 +606,11 @@
*
* #### Server Timing headers
*
* By default, we do not include Server Timing headers in the response. If you want to include them, you can pass an
* By default, we do not include Server Timing headers in responses. If you want to include them, you can pass an
* `includeServerTiming` option to the `createVerifiedFetch` function to include them in all future responses. You can
* also pass the `includeServerTiming` option to each fetch call to include them only for that specific response.
*
* @see https://github.com/ipfs/helia-verified-fetch/pull/164 for more information.
* See PR where this was added, https://github.com/ipfs/helia-verified-fetch/pull/164, for more information.
*
* ### Possible Scenarios that could cause confusion
*
Expand Down

0 comments on commit b410346

Please sign in to comment.