Throw exception when header is invalid

[michalbundyra]

Fixes #180 - infinite recursion

• Are you fixing a bug?
  • Detail how the bug is invoked currently.
  • Detail the original, incorrect behavior.
  • Detail the new, expected behavior.
  • Base your feature on the master branch, and submit against that branch.
  • Add a regression test that demonstrates the bug, and proves the fix.
  • Add a CHANGELOG.md entry for the fix.

[roypalacios]

Hi @webimpress,

It looks good to me. I'll only add few comments:

            // Generic Header should throw an exception if it fails
            if ($isGeneric) {
                throw $exception;
            }

            // Retry one more time with GenericHeader
            return $this->lazyLoadHeader($index, true);

Also, there is a minor optimization we can make at the beginning of the file:

If Generic is true, there is no need parse the key and call the ClassLoader.

I have an enhancement in 2.11.0, so if there's anything I can do to get there, please ask :)

[michalbundyra]

@roypalacios

Thanks for suggestions. I should be working on zend-http 'soon'. As you can see there is a lot of PR I've created recently and more is incoming. I just need to find a time to describe and merge all of them to issue new release. Thanks for your patience.