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

Missing headers and body in HTTP Request with Psr17 #7

Open
wants to merge 3 commits into
base: 6.x
Choose a base branch
from

Conversation

rudyzeinoun
Copy link

When switching to Psr17, the createRequest function only takes $method and $url. The headers and body have to be sent in separately after. This causes all POST requests to fail with permission error since the token is the first missing parameter that is checked.

When switching to Psr17, the createRequest function only takes $method and $url. The headers and body have to be sent in separately after. This causes all POST requests to fail with permission error since the token is the first missing parameter that is checked.
ashiquzzaman33 added a commit to sigma-smartcomm/facebook-php-graph-sdk that referenced this pull request Nov 12, 2023
@dpslwk
Copy link

dpslwk commented Dec 9, 2023

I think 6.0.1 is broken with this PR

after getting 6.0.1 in a composer update are logs are filling with this when trying to post to FB via joelbutcher/laravel-facebook-graph

Facebook\Exception\AuthenticationException: (#100) 0 does not resolve to a valid user ID in /home/forge/cryptologyrooms.co.uk/releases/20231209142658/vendor/joelbutcher/facebook-graph-sdk/src/Exception/ResponseException.php:101

and seems I might not be the only person effected

https://developers.facebook.com/community/threads/1350894655461720/

@primnym
Copy link

primnym commented Mar 26, 2024

I have the same issue, and it is resolved with this contribution. Can it be uploaded to the project?

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

Successfully merging this pull request may close these issues.

3 participants