innerHTML doesn’t seem to be a property of element

A

amorfati•5/31/22, 7:43 PM

the first argument of

on()

on()

is what the rewriter will read, everything else ignored. So if I do

HTMLRewriter().on('*', new ElementHandler())

HTMLRewriter().on('*', new ElementHandler())

then filter for element with tagName "article" inside the

ElementHandler

ElementHandler

class, it's the same as

HTMLRewriter().on('article', new ElementHandler())

HTMLRewriter().on('article', new ElementHandler())

?

A

amorfati•5/31/22, 7:44 PM

it's more computation passing

then filtering (is my understanding)

Kkian innerHTML doesn’t seem to be a property of element

A

amorfati•5/31/22, 7:49 PM

thanks, this helps. I think the rest of my problems is the fact that the middleware reader reads the html before the content has been fully loaded.

N

nitsky•5/31/22, 8:07 PM

Hi, is there any support for Rust+WASM in functions?

Ggarrison My function is somehow causing an Error 1101: Worker threw exception, but I don'...

J

James•6/1/22, 2:00 AM

Unfortunately there aren’t any error logs for Functions yet, sorry!

Does your function work locally with pages dev? If so, could you possibly share it here so we can try and help you diagnose why it’s not working in production?

Ggarrison Thanks for the info! I actually don't even know how to test it locally, but I wi...

J

James•6/1/22, 2:04 AM

I’d definitely recommend testing locally first then: https://developers.cloudflare.com/pages/platform/functions/#develop-and-preview-locally
That should give you a pretty good representation of how things will run and should let you diagnose any major issues

Functions (beta) · Cloudflare Pages docs

Documentation for Cloudflare Pages, the best way to deploy your static and JAMstack sites

Ggarrison I'm running it locally. Where should I expect an error log to appear? (In the ...

J

James•6/1/22, 2:13 AM

Console, if there are any errors

J

James•6/1/22, 2:13 AM

Is it working fine locally?

J

James•6/1/22, 2:15 AM

Can you share the function code/repo? That seems odd

Ggarrison https://github.com/garrison/cloudflare-pages-shared-password/blob/main/functions...

J

James•6/1/22, 2:22 AM

Thanks. Couple of immediate things: the errors you're throwing aren't really errors, so they're being logged weirdly - that's why you're seeing the

object Object

object Object

logs

J

James•6/1/22, 2:22 AM

Replace your errors with something like this (haven't solved any typescript things):

class UnauthorizedException extends Error {
  constructor(reason) {
    super(reason);
    this.status = 401;
    this.statusText = 'Unauthorized';
    this.reason = reason;
  }
}

class BadRequestException extends Error {
  constructor(reason) {
    super(reason);
    this.status = 400;
    this.statusText = 'Bad Request';
    this.reason = reason;
  }
}

class UnauthorizedException extends Error {
  constructor(reason) {
    super(reason);
    this.status = 401;
    this.statusText = 'Unauthorized';
    this.reason = reason;
  }
}

class BadRequestException extends Error {
  constructor(reason) {
    super(reason);
    this.status = 400;
    this.statusText = 'Bad Request';
    this.reason = reason;
  }
}

class UnauthorizedException extends Error {
  constructor(reason) {
    super(reason);
    this.status = 401;
    this.statusText = 'Unauthorized';
    this.reason = reason;
  }
}

class BadRequestException extends Error {
  constructor(reason) {
    super(reason);
    this.status = 400;
    this.statusText = 'Bad Request';
    this.reason = reason;
  }
}

class UnauthorizedException extends Error {
  constructor(reason) {
    super(reason);
    this.status = 401;
    this.statusText = 'Unauthorized';
    this.reason = reason;
  }
}

class BadRequestException extends Error {
  constructor(reason) {
    super(reason);
    this.status = 400;
    this.statusText = 'Bad Request';
    this.reason = reason;
  }
}

J

James•6/1/22, 2:23 AM

Then you'll get more expected errors I imagine, like I see

[pages:err] GET /: Error: Please use a HTTPS connection.

[pages:err] GET /: Error: Please use a HTTPS connection.

[pages:err] GET /: Error: Please use a HTTPS connection.

[pages:err] GET /: Error: Please use a HTTPS connection. for example.

J

James•6/1/22, 2:24 AM

If I comment out the

https

https

https

https detection stuff real quick, I can get through the basic auth without issue

J

James•6/1/22, 2:26 AM

Otherwise this looks good to me!

J

James•6/1/22, 2:26 AM

Oh wait, I see another potential issue that's super nuanced, one sec

J

James•6/1/22, 2:28 AM

const response = await next();
response.headers.set('Cache-Control', 'no-store');
return response;

const response = await next();
response.headers.set('Cache-Control', 'no-store');
return response;

const response = await next();
response.headers.set('Cache-Control', 'no-store');
return response;

const response = await next();
response.headers.set('Cache-Control', 'no-store');
return response;

^ This code will likely work in dev, but not in prod. In dev, headers from effective

fetch

fetch

requests can be mutable, whereas in prod they're not. There's various issues to resolve this (and I think it's mostly resolved now), but for safety you may want to clone the response before mutating the headers, like this:

const response = await next();
const newResponse = response.clone();
newResponse.headers.set('Cache-Control', 'no-store');
return newResponse;

const response = await next();
const newResponse = response.clone();
newResponse.headers.set('Cache-Control', 'no-store');
return newResponse;

const response = await next();
const newResponse = response.clone();
newResponse.headers.set('Cache-Control', 'no-store');
return newResponse;

const response = await next();
const newResponse = response.clone();
newResponse.headers.set('Cache-Control', 'no-store');
return newResponse;

J

James•6/1/22, 2:28 AM

Lemme know if all of that gets you any further along!

JJames ```js const response = await next(); response.headers.set('Cache-Control', 'no-s...

I

Isaac McFadyen•6/1/22, 2:29 AM

I don't think clone works, I think you need to do

new Response(response.body, response)

new Response(response.body, response)

new Response(response.body, response)

new Response(response.body, response)?

I

Isaac McFadyen•6/1/22, 2:29 AM

Maybe that's just for Request though.

J

James•6/1/22, 2:31 AM

Clone should work afaik, but that's another great tip if it doesn't

J

James•6/1/22, 2:32 AM

Once https://github.com/cloudflare/miniflare/commit/1599c75c087f04fcc1a969b93588d802052cf629 lands in wrangler2 though, things should hopefully be more consistent in prod vs dev

Ggarrison Yup, replaced clone() with what was suggested above, and now it works.

J

James•6/1/22, 3:07 AM

Glad to hear it

A

amorfati•6/1/22, 6:49 AM

in the code sample in this screenshot, if we were to apply the KV in the shown KV namespace bindings, would it be:

npx wrangler pages dev dist --kv appleData

npx wrangler pages dev dist --kv appleData

npx wrangler pages dev dist --kv appleData

npx wrangler pages dev dist --kv appleData

or

npx wrangler pages dev dist --kv APPLE_DISTRIBUTION

npx wrangler pages dev dist --kv APPLE_DISTRIBUTION

npx wrangler pages dev dist --kv APPLE_DISTRIBUTION

npx wrangler pages dev dist --kv APPLE_DISTRIBUTION

then:

await env.appleData.get('APPLE_DISTRIBUTION');

await env.appleData.get('APPLE_DISTRIBUTION');

await env.appleData.get('APPLE_DISTRIBUTION');

await env.appleData.get('APPLE_DISTRIBUTION');

or

await env.APPLE_DISTRIBUTION.get('...');

await env.APPLE_DISTRIBUTION.get('...');

await env.APPLE_DISTRIBUTION.get('...');

await env.APPLE_DISTRIBUTION.get('...');

or something else?

W

Walshy•6/1/22, 9:08 AM

number 2 and number 2

I

Isaac McFadyen•6/1/22, 3:03 PM

@jbergstroem are you seeing the same issue with any other colos?

J

jbergstroem•6/1/22, 3:05 PM

So far, I've tried from

7148cf4558a76f2f-SCL

7148cf4558a76f2f-SCL

7148cf4558a76f2f-SCL

7148cf4558a76f2f-SCL (boo),

7148c9e2de03ab38-EZE

7148c9e2de03ab38-EZE

7148c9e2de03ab38-EZE

7148c9e2de03ab38-EZE (yay) and

7148c2165e46694b-FRA

7148c2165e46694b-FRA

7148c2165e46694b-FRA

7148c2165e46694b-FRA (yay)

J

jbergstroem•6/1/22, 3:06 PM

We've also seen worker errors from

-SCL

-SCL

-SCL

-SCL; here's a ray id that threw a worker error (1101):

7148a5d07dc62ded

7148a5d07dc62ded

7148a5d07dc62ded

7148a5d07dc62ded

I

Isaac McFadyen•6/1/22, 3:06 PM

CC @walshydev a

_worker.js

_worker.js

_worker.js

_worker.js seems to be returning an empty 200 response in certain colos only (same deployment), Ray IDs are above.

W

Walshy•6/1/22, 3:08 PM

what's the site?

W

Walshy•6/1/22, 3:08 PM

was this a custom domain or pages.dev?

W

Walshy•6/1/22, 3:08 PM

what's the exact response returned?

I

Isaac McFadyen•6/1/22, 3:09 PM

@jbergstroem ^

J

jbergstroem•6/1/22, 3:12 PM

So, I just did a redeployment and the pages subdomain (

7148df54b9eefde9-SCL

7148df54b9eefde9-SCL

7148df54b9eefde9-SCL

7148df54b9eefde9-SCL) is confirmed working and the custom domain wasn't. The custom domain now responds from

7148e104883a9b39-FRA

7148e104883a9b39-FRA

7148e104883a9b39-FRA

7148e104883a9b39-FRA (working fine), checking with other users.

J

jbergstroem•6/1/22, 3:14 PM

@walshydev basically a lot of the linked assets are returning a http 200 with empty content. I'm deploying my site via sveltekit's cloudfare pages adapter which uses the

_worker.js

_worker.js

_worker.js

_worker.js approach.

J

jbergstroem•6/1/22, 3:14 PM

I occasionally saw 1101 errors as well.

W

Walshy•6/1/22, 3:15 PM

1101 is a Worker error

W

Walshy•6/1/22, 3:15 PM

It wouldn't surprise me if this was just an issue with the Function itself

W

Walshy•6/1/22, 3:15 PM

I'd recommend getting some logging and seeing what turns up

J

jbergstroem•6/1/22, 3:16 PM

Yep; just saying that I got inconsistent response from different datacenters.

J

jbergstroem•6/1/22, 3:16 PM

As far as logging goes, you mean that I should deploy it as a worker instead?

innerHTML doesn’t seem to be a property of element

Similar Threads

Similar Threads

Similar Threads