Cloudflare Developers•3y ago

esbuild will try to eliminate dead code - <https://esbuild.github.io/api/#tree-shaking>

esbuild will try to eliminate dead code - https://esbuild.github.io/api/#tree-shaking

kianOP•2/27/23, 3:52 PM

It's still orders of magnitude larger than aws4fetch

Kkian esbuild will try to eliminate dead code - <https://esbuild.github.io/api/#tree-s...

rdutton•2/27/23, 3:52 PM

Thanks, good to know

Hello, I’m Allie!•2/28/23, 9:32 AM

What's the best way to have console.logconsole.logs in JS code that are automatically stripped out by esbuildesbuild, unless some specific action is performed during the build?

HHello, I’m Allie!What's the best way to have `console.log`s in JS code that are automatically str...

Hello, I’m Allie!•2/28/23, 10:10 AM

Wrote a plugin for it:

const devRetainer = {
  name: "devRetainer",
  setup(build) {
    build.onLoad({ filter: /\.ts$/ }, async (args) => ({
      contents: (await readFile(args.path, "utf8")).replace(/(\/\* Debugging Statement Start)|(Debugging Statement End \*\/)/g, ""),
      loader: "ts",
    }));
  },
};

const devRetainer = {
  name: "devRetainer",
  setup(build) {
    build.onLoad({ filter: /\.ts$/ }, async (args) => ({
      contents: (await readFile(args.path, "utf8")).replace(/(\/\* Debugging Statement Start)|(Debugging Statement End \*\/)/g, ""),
      loader: "ts",
    }));
  },
};

Cyb3r-Jak3•3/1/23, 1:39 AM

Zone -> Workers Routes -> Edit maybe?

Chaika•3/1/23, 2:59 AM

hmm can't find anything in the docs about it and Discord search is still broken blobnotlikethis

I wonder if it's not Zone -> DNS -> Edit, since they show as DNS Records / are "sort of" DNS Level

Chaika•3/1/23, 3:09 AM

Account id / zone ids are fine to be public

Cyb3r-Jak3•3/1/23, 3:09 AM

^ All ids are just uuid4, account, zone, KV, D1 etc

zegevlier•3/1/23, 11:23 AM

What do you mean with 7000 routers?

Hello, I’m Allie!•3/1/23, 11:36 AM

I forgot, can’t you do /?

Hello, I’m Allie!•3/1/23, 11:36 AM

Oop

Hello, I’m Allie!•3/1/23, 11:36 AM

I meant */*?

Hello, I’m Allie!•3/1/23, 11:38 AM

With the amount of routes, it might be easier to manually exclude the handful of routes they don’t need…

Original message was deleted

Erisa•3/1/23, 11:46 AM

It wont, routes are scoped to zones still

Erisa•3/1/23, 11:46 AM

you need one route per zone, what theyre doing is terrifying but correct

Erisa•3/1/23, 11:47 AM

unfortunately, having too many routes causes wrangler to break with a timeout - you will have to completely omit routesroutes from the wrangler.toml and it should leave alone the ones on the dashboard @NeMetS_666 which you can configure there

Walshy•3/1/23, 11:49 AM

Note adding thousands of routes will impact performance. You should work to reduce that

Erisa•3/1/23, 11:49 AM

Would it if theyre all on different zones?

Erisa•3/1/23, 11:49 AM

e.g. the worker has thousands but each zone only has 1

Walshy•3/1/23, 11:51 AM

Good question… I’m too used to Pages routing tables.
I’d need to verify

Rico.•3/1/23, 11:59 PM

Hi all, I'm using Ghost, and a lot of the HTML can't be edited. What I want to do is to create an interstitial with Turnstile when a user hits the /ghost path (which is the login page), but it'll be a GET request not POST, unless I actually login, but I can't do that cause I can't edit he HTML to add Turnstile. How do I go about doing this? I have the code here: https://github.com/erfianugrah/turnstile-interstitial

GitHub

GitHub - erfianugrah/turnstile-interstitial: turnstile-interstitial

turnstile-interstitial. Contribute to erfianugrah/turnstile-interstitial development by creating an account on GitHub.

Rico.•3/1/23, 11:59 PM

I can't seem to redirect users back to the login portal, it just gets stuck on the interstitial

Rico.•3/1/23, 11:59 PM

Any pointers?

Erisa•3/2/23, 12:00 AM

if you need an interstitial, couldn't you put a Managed Challenge on the url instead?

Rico.•3/2/23, 12:01 AM

I could do that, but part of this exercise was to extend Ghost in some capacity and use Turnstile, even though it might seem convoluted haha

Erisa•3/2/23, 12:01 AM

no worries - first thing that comes to mind is running a Worker on the route and using it to modify the response html on the fly

Rico.•3/2/23, 12:02 AM

The signin divs are all SSRs, so I can't use HTMLrewriter on them unless top most div

Rico.•3/2/23, 12:02 AM

yeah ghost is a tough nut

Rico.•3/2/23, 12:02 AM

haha

Rico.•3/2/23, 12:02 AM

I see the code on their monorepo but it's not something I can edit

Erisa•3/2/23, 12:02 AM

I use it myself but i just threw Access in front of the login, aha

esbuild will try to eliminate dead code - <https://esbuild.github.io/api/#tree-shaking>

Similar Threads

Similar Threads

Similar Threads