search

ChartBlocks / php-ssrs

PHP library for connecting to SSRS over SOAP
MIT License
25 stars 22 forks source link

Content-Length header must not be set #10

Closed SmokeyBrandie closed 9 years ago

SmokeyBrandie commented 9 years ago

The content length hader must not be set since CURL sets the length automatically

So I suggest to remove the following line: $headers[] = 'Content-Length: ' . strlen($data);

From function generateHeaders in File NTLM.php

rb-cohen commented 9 years ago

Pushed up to master. Out of interest, was the calculated value incorrect, or were you getting two headers in the request?

rb-cohen commented 9 years ago

P.S. Happy to accept pull requests if you'd like to fork the lib and make changes faster than I can get back to you.