molenzwiebel
04/21/2021, 2:56 PMmolenzwiebel
04/21/2021, 2:57 PMDeleted User
04/21/2021, 3:56 PMHardAtWork
04/21/2021, 4:51 PMmatt
04/21/2021, 4:54 PMnode_modules
and minify your codematt
04/21/2021, 4:54 PMmolmorg
04/21/2021, 5:01 PMmolmorg
04/21/2021, 5:01 PMmolenzwiebel
04/21/2021, 5:03 PMmolmorg
04/21/2021, 5:11 PMjohn.spurlock
04/21/2021, 6:03 PMwaitUntil
is available), and noticed that the incoming request.cf
is not available for module workers, even for non-DO calls. Is this a known issue?kenton
04/21/2021, 6:45 PMcf
is delivered and so didn't want to make it available in modules yet... but I think we might give up and leave it as it is.john.spurlock
04/21/2021, 7:04 PMkenton
04/21/2021, 7:06 PMkenton
04/21/2021, 7:07 PMcf
somewhere else, like onto the new "context" object.kenton
04/21/2021, 7:09 PMfetch
, like scheduled
kenton
04/21/2021, 7:09 PMrequest.cf
is, especially in the case of a multi-worker pipeline.kenton
04/21/2021, 7:09 PMrequest.cf
before sending it on to the next worker?kenton
04/21/2021, 7:10 PMkenton
04/21/2021, 7:11 PMcf
. But is that... right? What if one worker in the pipeline wants to hide some information from later workers? It's hard to think about this when the use cases aren't clear yet.john.spurlock
04/21/2021, 7:18 PMscheduled
, it's clear - I noticed that the context arg is already being passed as the 3rd arg, and the incoming cf props would be unambiguous there. By multi-worker pipeline, do you mean workers calling DOs via fetch? or workers calling other workers? (across zones). For the latter, shouldn't cf
always be clear (should refer to the one being called)? So we're really talking about DO fetches. In that case, the same "caller info" referring the calling request is the most unambiguous, any n-level call stack should be probably left to the app via a correlation id etc.kenton
04/21/2021, 7:20 PMkenton
04/21/2021, 7:21 PMjohn.spurlock
04/21/2021, 7:25 PMcf
for the incoming request makes the mental model easier, even when multiple workers are in play. Not sure what the value of a Laplace's demon worker changing the colo / location info for downstream workers is, but maybe you have some crazy internal worker scenarios : )kenton
04/21/2021, 7:28 PMcf
is used for all kinds of things. It's really just "additional headers that won't get transmitted outside Cloudflare". So rewriting it is like rewriting headers, and obviously rewriting headers is useful for all sorts of stuff...kenton
04/21/2021, 7:28 PMcf
contains a lot of info about the original request received by our edge, but passing such information down through several layers of workers might be breaking layers of abstractionjohn.spurlock
04/21/2021, 7:33 PMvans163
04/21/2021, 7:49 PMvans163
04/21/2021, 7:49 PMconstructor(controller, env, ctx) {
vans163
04/21/2021, 7:51 PM