Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bad Request when method is specified in lowercase #108

Closed
jimyaghi opened this issue Aug 24, 2016 · 1 comment
Closed

Bad Request when method is specified in lowercase #108

jimyaghi opened this issue Aug 24, 2016 · 1 comment

Comments

@jimyaghi
Copy link

When making a request using, say, the 'patch' method, if specified in lowercase it returns BAD REQUEST. If changed to uppercase it works.

Fix is simple. Line 118 in Vimeo.php needs to also use strtoupper, (not just the switch/case block)

$curl_opts = array(
CURLOPT_POST => true,
CURLOPT_CUSTOMREQUEST => strtoupper($method),
CURLOPT_POSTFIELDS => $body
);

@jimyaghi jimyaghi changed the title Bad Request because method is not uppercase Bad Request when method is specified in lowercase Aug 24, 2016
@Dashron
Copy link
Contributor

Dashron commented Sep 7, 2016

Nice catch, we'll get this added!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants