ldp
ldp
Explore posts from servers
CDCloudflare Developers
Created by ldp on 2/24/2024 in #workers-for-platforms
TLDR: `new ReadableStream()` is working
makes a lot of sense! thanks for the example this makes it even more surprising that compatibility_date isn't required when uploading user workers though
5 replies
CDCloudflare Developers
Created by ldp on 2/24/2024 in #workers-for-platforms
TLDR: `new ReadableStream()` is working
can you explain a bit more? not sure i understand how that's the case. as someone using the system, personally i find it really unintuitive. if you required an explicit compatibility_date with every user worker upload, then i would understand the thought process, but right now the compatibility_date on user workers is totally optional, as far as i can tell (not sure though bc of lack of documentation). it's already using a hidden default for user workers, i'm assuming? i didn't set one previously and they were working, just with different available functionality. i'm just proposing that if there is a hidden default anyway, it should prob be the same one as the wfp codebase to avoid that confusion in the future. or just require that compatibility_date is set with every user worker upload.
5 replies
CDCloudflare Developers
Created by ldp on 2/24/2024 in #workers-for-platforms
TLDR: `new ReadableStream()` is working
Ok thanks a lot kian, this led me in the right direction! I was able to figure it out after some trial and error. I created a PR for the WFP documentation here , not really sure how I would have known about the ability to configure this without guessing. Imo, it would be a bit more intuitive if the user workers inherited the compatibility_date from the WFP project's wrangler.toml, if it's not explicitly set when uploading the user worker
5 replies
CDCloudflare Developers
Created by ldp on 2/24/2024 in #workers-for-platforms
TLDR: `new ReadableStream()` is working
The code that is erroring can be found on this npm beta package Looks like you can't link directly to files on npm, but the line causing the error is at/dist/esm/PulseSubscription.workerd.js on line 87
function createWebSocketStream(webSocket, signal) {
return new ReadableStream({
// ...
function createWebSocketStream(webSocket, signal) {
return new ReadableStream({
// ...
5 replies
CDCloudflare Developers
Created by ldp on 2/21/2024 in #workers-help
workers deployment down?
ty ❤️
5 replies
CDCloudflare Developers
Created by ldp on 2/21/2024 in #workers-help
workers deployment down?
fastest response ive ever gotten from any CS ever, lol
5 replies