Skip to content

Commit ea1ca5d

Browse files
authored
Merge pull request rails#43461 from ignacio-chiazzo/update-changelog
Update missing actionpack CHANGELOG entry for wildcard route fix
2 parents eee0c03 + 1fb98c8 commit ea1ca5d

File tree

1 file changed

+19
-0
lines changed

1 file changed

+19
-0
lines changed

actionpack/CHANGELOG.md

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,22 @@
1+
* Allow multiline to be passed in routes when using wildcard segments.
2+
3+
Previously routes with newlines weren't detected when using wildcard segments, returning
4+
a `No route matches` error.
5+
After this change, routes with newlines are detected on wildcard segments. Example
6+
7+
```ruby
8+
draw do
9+
get "/wildcard/*wildcard_segment", to: SimpleApp.new("foo#index"), as: :wildcard
10+
end
11+
12+
# After the change, the path matches.
13+
assert_equal "/wildcard/a%0Anewline", url_helpers.wildcard_path(wildcard_segment: "a\nnewline")
14+
```
15+
16+
Fixes #39103
17+
18+
*Ignacio Chiazzo*
19+
120
* Treat html suffix in controller translation.
221

322
*Rui Onodera*, *Gavin Miller*

0 commit comments

Comments
 (0)