setcookie() is passed a null instead of 0/[] during OAuth Callback.

[da-mask]

Issue summary

Write a short description of the issue here ↓
When using the package for OAuth of an offline/not embedded app, the function OAUTH::callback() attempts to use setcookie() with a null value for expires: parameter.

Expected behavior

What do you think should happen?
Either a 0 or empty array [] should be passed in instead. We could either make sure expiration does not fallback to being null, or change the constructor of OAuthCookie to set its own value to 0 if null is passed in.

Actual behavior

What actually happens?

setcookie(): Argument #3 ($expires_or_options) must be of type array|int, null given

Steps to reproduce the problem

1. OAuth::begin( shop: $this->shop->url, redirectPath: $this->redirectUri, isOnline: false, );
2. $session = OAuth::callback( cookies: $cookies, query: $get ); $accessToken = $session->getAccessToken();

---

Checklist

• I have described this issue in a way that is actionable (if possible)

[github-actions]

This issue is stale because it has been open for 60 days with no activity. It will be closed if no further action occurs in 14 days.

[github-actions]

We are closing this issue because it has been inactive for a few months.
This probably means that it is not reproducible or it has been fixed in a newer version.
If it’s an enhancement and hasn’t been taken on since it was submitted, then it seems other issues have taken priority.

If you still encounter this issue with the latest stable version, please reopen using the issue template. You can also contribute directly by submitting a pull request– see the CONTRIBUTING.md file for guidelines

Thank you!

[erika89]

Hi @da-mask , did you solve this issue??? It would be useful for me to know the answer, as i'm having the same problem.

[da-mask]

Hey @erika89 , I can't really remember. I'm guessing you've looked at my associated PRs? Were they any help?

[erika89]

Yes @da-mask , i saw the PR after commenting this. I manage to solve it by doing the same, and forgot to let you know hehe.
Thanks a lot!!!!