coverallsapp / coverallsapp/github-action
Change 504 error page/handling to not dump 3/4mb of content (700k of which is base64 noise)
Nobody has claimed this yet.
- Dominant language
- No language data
- Stars
- 520
- Forks
- 78
- PR merge metrics
- No merged PRs in 30d
Description
Please consider changing your 504 error page to not embed 5 woff, 2 eot, and 1 truetype fonts as data: urls. The page is 3/4mb. >1/4mb of that is fonts (plus 250kb for an svg, and another ~75k for two pngs). The entire response is dumped by coverallsapp/github-action@v2 when hit, as in https://github.com/PowerDNS/pdns/actions/runs/10411289118/job/29294135610#step:2:85
While you're fixing the 504 page (by removing the 700kb of noise), could you change it to actually say Five Zero Four -- it might be hard to notice in the noise, but it's a little easier with the noise omitted:
<body id="errorPage">
<div class="cf-error-details cf-error-504">
<h1>Gateway time-out</h1>
<p>The web server reported a gateway time-out error.</p>
<ul>
<li>Ray ID: 8b9a6d7d1fd32ab7</li>
<li>Your IP address: 20.172.5.45</li>
<li>Error reference number: 504</li>
<li>Cloudflare Location: Los Angeles</li>
</ul>
</div>
...
<div class="container site-wrapper">
<h1>
<img alt="Public_stomp" src="">
Five Zero Zero
</h1>
<div class="alert alert-error">
<h3>It's Gone Pear Shaped</h3>
<p>There has been a hiccup, the revolution will continue after we stamp out the refusniks. You could also try refreshing or returning to the home page.</p>
</div>
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the 504 response produced when coverallsapp/github-action@v2 is hit and determine whether the error page is maintained in this repository. Compare the response with the issue's requested compact page, then verify that the 504 output no longer embeds the listed fonts and large image data and displays the requested error wording.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions
- Domain
- ci-cd
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 30/100