Fix getallheaders support for FastCGI and other servers then Apache

[GoogleCodeExporter]

Fix getallheaders support for FastCGI and other servers then Apache

[GoogleCodeExporter]

Thanks a lot! I added it to the SVN, with two slight changes:

- added to OAuthRequestLogger instead
- apache_request_headers() is returned as is, without passing through the 
foreach loop.

Unless you have something to add, I'll consider this fixed.

Original comment by brunobg%[email protected] on 5 Mar 2010 at 10:12

• Changed state: Started

[GoogleCodeExporter]

Well I wouldn't expect that function in OAuthRequestLogger, because it's not a
logging function, but an utility function used to extract headers from the 
webserver.
So in my opinion I shouldn't add it to the Logger class. I added it the 
OAuthRequest,
because there are the other utility functions as well, as parse_url and stuff 
like
that. So that's where I should expect it.

The apache_request_headers() can be returned as is yes. But please add there a 
ksort
as well. So you will get the same sorting order as with $_SERVER and $_ENV. 
This will
minimize the differences between the 2 methods.

Besides this, I consider it fixed as well.

Original comment by [email protected] on 6 Mar 2010 at 9:06

[GoogleCodeExporter]

I agree that Logger is not the most logical place, but it's the lowest common
dependency and it also uses this function. I added the ksort and closing this. 
Thanks
again for the patch.

Original comment by brunobg%[email protected] on 8 Mar 2010 at 12:46

• Changed state: Fixed