Closed andrelaszlo closed 2 years ago
@ncr Is this useful? Would like to use rack-proxy but we need those security patches first. :) I'm available to fix feedback on Friday and any time next week.
(Sorry for the force-pushes; I needed to get author/committer to match my key or Github wouldn't verify it properly.)
Hello @andrelaszlo, thank you for this contribution. I will merge it and release a new version shortly :)
Fixes #97 Fixes #63
master
that I also fixed. They were referring to various domains, so I pointed them to a "neutral" one - mockapi.io. I have no affiliation with them.assert x==y
→assert_equal x, y
Set-Cookie
header. (See #63) HTTP headers are case insensitive, butRack::MockResponse
is callingheaders.fetch
which isn't case insensitive even whenheaders
is aUtils::HeaderHash
instance. The easiest solution seemed to be to just change our normalized header format to whateverRack::MockResponse
expects.