Skip to content

Fix local resource URLs with hashes#8589

Open
maxrross wants to merge 1 commit into
4ian:masterfrom
maxrross:codex/fix-file-url-hash-preview
Open

Fix local resource URLs with hashes#8589
maxrross wants to merge 1 commit into
4ian:masterfrom
maxrross:codex/fix-file-url-hash-preview

Conversation

@maxrross

Copy link
Copy Markdown

Fixes #4015.

Summary

  • Escape local Electron file URLs so # and ? in resource paths are treated as path characters instead of URL fragments/query markers.
  • Decode escaped file URLs when turning resource URLs back into local paths.
  • Add regression coverage for hashed resource paths and decoded local paths.

Tests

  • npm --prefix newIDE/app test -- --runTestsByPath src/ResourcesLoader/index.spec.js src/ResourcesList/ResourceUtils.spec.js --watchAll=false
@maxrross maxrross requested a review from 4ian as a code owner May 11, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant