phpfui / ConstantContact

MIT License
16 stars 7 forks source link

http_build_param Issue #16

Closed iamseventhpro closed 1 year ago

iamseventhpro commented 1 year ago

When tryin to authenticate, the authentication url uses &instead of just &. Tried both approach and the url with just "&" is the one working.. & returns 400 error.

After some tests, seems that it only happens when using the php head redirect (\header('location: ' . $client->getAuthorizationURL());)

Im not sure if this is related to that but when getting the access_toke, I don't get results. I have the code and state in url but still no access_token or refresh token after calling $client->acquireAccessToken($_GET); object(PHPFUI\ConstantContact\Client)#22 (17) { ["accessToken"]=> string(0) "" ["refreshToken"]=> string(0) "" ["authorizeURL":"PHPFUI\ConstantContact\Client":private]=> string(61) "https://authz.constantcontact.com/oauth2/default/v1/authorize"

phpfui commented 1 year ago

Looks like an HTLM entity encode problem. Do you have source code that will reproduce the problem.

I can fix today if it is a bug in the library.

phpfui commented 1 year ago

Looking at the code, I don't see any HTML Entity encoding. Try printing out the return from authorizeURL() and see if it is correct. You may be encoding it somewhere else accidentally.

iamseventhpro commented 1 year ago

It seems that we have & -& conversion in our settings. Disabled that. Working fine now.

phpfui commented 1 year ago

Thanks for the followup. Let me know of any other problems.