Add multi mapping support via header#1187
Conversation
|
I do not believe this has made it into any release yet - any plans to merge into https://github.com/rack/rack/blob/2.0.6/lib/rack/sendfile.rb#L153 https://github.com/rack/rack/blob/2.0.7/lib/rack/sendfile.rb#L153 https://github.com/rack/rack/blob/2.0.8/lib/rack/sendfile.rb#L153 |
We need to support multiple mappings for X-Accel-Mapping nginx header. The original PR on the original repo ([PR rack#1187](https://github.com/rack/rack/pull/1187/files)) has been merged onto master but not released yet.
|
Seemingly this duplicates #853, although it uses commas instead of semicolons for the separator. I guess the other PR should be closed? |
|
Those two PRs are really similar indeed. I also think PR #853 could then be closed since the current PR we're commenting on has already been merged into master. |
|
I'm wondering why don't release it? |
nignx sets "HTTP_X_ACCEL_MAPPING" from multi mappings to comma splited string.
like,
to