Codeberg-Infrastructure/forgejo
26
177
Fork 38
Code Issues Pull requests 6 Releases Activity

CB/perf: serve logo from Forgejo assets #131

Open
0ko wants to merge 2 commits from 0ko/cb-logo-sametls into codeberg-14 AGit
pull from: 0ko/cb-logo-sametls
merge into: Codeberg-Infrastructure:codeberg-14
Conversation 0 Commits 2 Files changed 7 +336 -6
0ko commented 2026-03-01 18:45:15 +01:00
Contributor
Copy link

Followup to !117

Currently the logo icons are being served unoptimized, and from a separate domain, which requires browser to perform another TLS handshake before it is able to download the logo, after it discovers the logo on the page.

So now it should be faster to download, and the alt text will be displayed for much less time.

Forgejo is already serving many assets, including it's own wordmark for the footer, so it shouldn't cause more load on the Forgejo server.

Followup to !117 Currently the logo icons are being served unoptimized, and from a separate domain, which requires browser to perform another TLS handshake before it is able to download the logo, after it discovers the logo on the page. So now it should be faster to download, and the alt text will be displayed for much less time. Forgejo is already serving many assets, including it's own wordmark for the footer, so it shouldn't cause more load on the Forgejo server.
Currently the logo icons are being served unoptimized, and from a separate domain, which requires browser to perform another TLS handshake before it is able to download the logo, after it discovers the logo on the page.
0ko requested reviews from Beowulf, Gusted 2026-03-01 18:45:15 +01:00
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin +refs/pull/131/head:0ko/cb-logo-sametls
git switch 0ko/cb-logo-sametls

Merge

Merge the changes and update on Forgejo.
git switch codeberg-14
git merge --no-ff 0ko/cb-logo-sametls
git switch 0ko/cb-logo-sametls
git rebase codeberg-14
git switch codeberg-14
git merge --ff-only 0ko/cb-logo-sametls
git switch 0ko/cb-logo-sametls
git rebase codeberg-14
git switch codeberg-14
git merge --no-ff 0ko/cb-logo-sametls
git switch codeberg-14
git merge --squash 0ko/cb-logo-sametls
git switch codeberg-14
git merge --ff-only 0ko/cb-logo-sametls
git switch codeberg-14
git merge 0ko/cb-logo-sametls
git push origin codeberg-14
Sign in to join this conversation.
Reviewers
No reviewers
Beowulf
Gusted
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Codeberg-Infrastructure/forgejo!131
No description provided.