fix(router): re-encode decoded rest segment for 3-segment proxy paths #67

Closed

zaelgohary wants to merge 1 commit from development_fix_proxy_path_reencode_on_decoded_rest into development

zaelgohary commented

2026-04-28 08:38:47 +00:00

Member

Summary

OnlyOffice emits cache URLs with _ percent-encoded as %5f, which broke the previous strip-the-literal-prefix approach and routed requests to /admin/. Now uses the decoded rest from axum and re-encodes per path segment.

Changes

Replaced literal-prefix-strip with per-segment urlenc on the decoded rest
Added a tiny RFC 3986 unreserved-set urlenc helper (no new dep)

Test Results

Live curl on herodev: /cell/edit/Hello.xlsx and Arabic-named editor URL both 200 · playwright office_editor_real 3/3 ran (1 helper-skipped, none failed).

## Summary OnlyOffice emits cache URLs with `_` percent-encoded as `%5f`, which broke the previous strip-the-literal-prefix approach and routed requests to `/admin/`. Now uses the decoded `rest` from axum and re-encodes per path segment. ## Changes - Replaced literal-prefix-strip with per-segment urlenc on the decoded `rest` - Added a tiny RFC 3986 unreserved-set urlenc helper (no new dep) ## Test Results Live curl on herodev: `/cell/edit/Hello.xlsx` and Arabic-named editor URL both 200 · playwright office_editor_real 3/3 ran (1 helper-skipped, none failed).