Fix debugging logs not including headers for CF API requests and responses#10425
Fix debugging logs not including headers for CF API requests and responses#10425dario-piotrowicz merged 2 commits intomainfrom
Conversation
🦋 Changeset detectedLatest commit: b67a699 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
|
Failed to automatically backport this PR's changes to Wrangler v3. Please manually create a PR targeting the Depending on your changes, running Notes:
|
create-cloudflare
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
wrangler
commit: |
|
I think the debugging logs could use some more tests (for example I don't think we have any tests for debug logs for api responses), but just not to go out of scope here I just added a simple check to make sure that headers are indeed included in the non sanitized debug logs |
|
Note I've added the |
Co-authored-by: Carmen Popoviciu <cpopoviciu@cloudflare.com>
Fixes #10421