[PR #150] [MERGED] fix: have caddy apply url prefix to relative redirects from services #174

Closed
opened 2026-02-16 12:51:43 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/stoatchat/self-hosted/pull/150
Author: @Docteh
Created: 5/13/2025
Status: Merged
Merged: 7/18/2025
Merged by: @insertish

Base: masterHead: rewriteresponse


📝 Commits (1)

  • fa230cb fix: have caddy apply url prefix to relative redirects from services

📊 Changes

1 file changed (+12 additions, -4 deletions)

View changed files

📝 Caddyfile (+12 -4)

📄 Description

…g with /

Definitely required for autumn, as /autumn/attachments/🆔 will redirect to /attachments/🆔/:filename: But we want the browser to redirect to /autumn/attachments/🆔/:filename:

Speculatively added for the other services

Please make sure to check the following tasks before opening and submitting a PR

  • I understand and have followed the contribution guide
  • I have tested my changes locally and they are working as intended

fixes #137


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/stoatchat/self-hosted/pull/150 **Author:** [@Docteh](https://github.com/Docteh) **Created:** 5/13/2025 **Status:** ✅ Merged **Merged:** 7/18/2025 **Merged by:** [@insertish](https://github.com/insertish) **Base:** `master` ← **Head:** `rewriteresponse` --- ### 📝 Commits (1) - [`fa230cb`](https://github.com/stoatchat/self-hosted/commit/fa230cba4e1c4f4d3f5d030efad72a997712892a) fix: have caddy apply url prefix to relative redirects from services ### 📊 Changes **1 file changed** (+12 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `Caddyfile` (+12 -4) </details> ### 📄 Description …g with / Definitely required for autumn, as /autumn/attachments/:id: will redirect to /attachments/:id:/:filename: But we want the browser to redirect to /autumn/attachments/:id:/:filename: Speculatively added for the other services ## Please make sure to check the following tasks before opening and submitting a PR - [x] I understand and have followed the [contribution guide](https://github.com/revoltchat/.github/blob/master/.github/CONTRIBUTING.md) - [x] I have tested my changes locally and they are working as intended fixes #137 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 12:51:43 -05:00
yindo closed this issue 2026-02-16 12:51:43 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: stoatchat/self-hosted#174