You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Because of strange implementation of URL class inside workers runtime, non-encoded parameters get encoded after toString invocation. It's not reproduced in Chrome/Firefox/node.js though.
I shall probably report an issue with it for Cloudflared - but can we review this function overall from the usage perspective as well?
Is there some reason to do URL construction on request clone? Can we have smth like:
downstreamRequest = new Request(request);
or
function cloneRequest(request) {
return new Request(request.url, request);
}
Thanks!
The text was updated successfully, but these errors were encountered:
Hi @RichiCoder1!
I am looking at the function
cloneRequest
:opentelemetry-sdk-workers/packages/opentelemetry-sdk-workers/src/utils.ts
Line 3 in 271b1b2
Because of strange implementation of URL class inside workers runtime, non-encoded parameters get encoded after
toString
invocation. It's not reproduced in Chrome/Firefox/node.js though.I shall probably report an issue with it for Cloudflared - but can we review this function overall from the usage perspective as well?
Is there some reason to do URL construction on request clone? Can we have smth like:
or
Thanks!
The text was updated successfully, but these errors were encountered: