Skip to content

Resolve local and quoted CSS asset URLs safely - #557

Closed
OskarEichler wants to merge 4 commits into
rails:masterfrom
OskarEichler:codex/resolve-data-directory-css-assets
Closed

OskarEichler wants to merge 4 commits into
rails:masterfrom
OskarEichler:codex/resolve-data-directory-css-assets

Conversation

@OskarEichler

@OskarEichler OskarEichler commented Aug 28, 2026 •

Copy link
Copy Markdown

Summary

Verification

  • cumulative unmodified suite: 151 tests / 796 assertions on Ruby 4.0.6 + Rails 8.1/Sprockets 4, Ruby 3.2.11 with the same stack, and Ruby 3.2.11 + Rails 7.2/Sprockets 3
  • cumulative external models: 28 checks per configuration, including quoted spaces, parentheses, schemes, fragments and local paths
  • syntax, package parity, bundle and backend integration checks pass

The earlier credited #543-derived commit and its existing regression tests remain on this branch. This follow-up changes runtime source only; no test file was changed in this audit pass.

OskarEichler and others added 4 commits August 28, 2026 10:27
Revives rails#543, closed by its author after adopting a workaround. Reuses the original runtime change; existing tests and external repros are run without editing test files.

Co-authored-by: Mike Voets <mike.voets@optimalworkshop.com>
@OskarEichler OskarEichler changed the title Resolve CSS assets in data and http-prefixed paths Resolve local and quoted CSS asset URLs safely Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants