-
Notifications
You must be signed in to change notification settings - Fork 76
Make request header blacklist case insensitive #202
Make request header blacklist case insensitive #202
Conversation
thewilkybarkid
commented
Mar 30, 2017
Q | A |
---|---|
Bug fix? | yes |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Tests pass? | yes |
Fixed tickets | |
License | Apache License 2.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like soon we will need separate method for headers preparation.
Actually, this is a BC break, as it changes the file naming strategy. |
a2c2226
to
4ee31a9
Compare
Ok, I've changed it so it tries normalising the request headers first, and falls back to not normalising. |
We're nearly there! Could you add a test to check that both the new and the old naming strategies return the expected paths? Thanks! 😃 |
@@ -66,8 +66,9 @@ public function testFetch() | |||
public function testSave() | |||
{ | |||
$request = $this->getRequestMock(); | |||
$mockStorage = new $this->class($this->tmpDir, [], ['X-Foo']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test should not be altered.
On the other hand, we should create tests which test specifically the naming strategies.
I'm closing this PR as the logic has been moved to a third-party library. @thewilkybarkid Want to make a PR against the new project? (https://github.com/csarrazi/guzzle-cache-middleware) :) |