Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

- #1

Closed
cuj opened this issue Sep 6, 2021 · 6 comments
Closed

- #1

cuj opened this issue Sep 6, 2021 · 6 comments

Comments

@cuj
Copy link

cuj commented Sep 6, 2021

@cirosantilli
Copy link
Owner

cirosantilli commented Sep 6, 2021

I saw the 500 now, it is quite intermittent.

= h
:var: abc

{var}

in asciidoctor should render as:

<h1>h

<p>abc

so I was trying to create a huge variable to see if things blow up.

Github runs Asciidoctor quite natively, so any adoc blowups lead to 500s, e.g. a previous one; https://twitter.com/cirosantilli/status/1283339613814370304

I forgot why these specific sizes were chosen. You can't just pick arbitrarily large because github cuts up file rendering at 512 KB: isaacs/github#1610 so pointless going beyond that.

@cirosantilli
Copy link
Owner

cirosantilli commented Sep 6, 2021

OK, remembered now, it was close to optimum, you have to optimize f(x, y) with x + 3y = 512KiB. Exact result is x = 256KiB and y = 512Kib/6. Previous script just did 512Kib/8to stay in power of 2. I got it closer to the limit now with85 * (2 ** 10)`.

We have to see if it is possible to add HTML in the variable. Then we can make the output much larger even. ( as it stands, out of memory killer already kills it on my Linux machine). Edit, didn't work:

= h
:a: *c*

{a}{a}{a}

does not expand * into <i>.

@cirosantilli
Copy link
Owner

cirosantilli commented Sep 6, 2021

(I don't understand why sometimes GitHub does not try to render and shows just the plaintext and does not blow up. For some reason 2 ** 15 blows more often.)

@cuj cuj changed the title And a pink unicorn shows up when loading this repository... - Jul 1, 2024
@cuj
Copy link
Author

cuj commented Jul 1, 2024

Hi cirosantilli, please delete this issue. closing is not enough. I want my issue to be deleted. Thank you

@cuj cuj closed this as completed Jul 1, 2024
@cuj
Copy link
Author

cuj commented Jul 1, 2024

And please delete all of my issues under cirosantilli/test-git-web-interface. Thanks @cirosantilli

@cirosantilli
Copy link
Owner

Can I ask why? Why not delete account and re-create it instead with same handle for example, so that everything goes to ghost?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants