-
Notifications
You must be signed in to change notification settings - Fork 30
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
Random failed authentication #26
Comments
Problems like that are hard to debug. It could be connectivity issues, like DB or network. Best thing to do is to hack the code and add some logging in places and see how far it gets. |
Hi, When you call it from login function, pass the $newSession parameter as true and when called from other functions, set it to default to false. A link to the curl documentation: |
Thanks for sharing. Do you plan to migrate to the new REST API (https://github.com/phpList/rest-API) at some stage? |
Thanks for your comment Sam. Right now, I am personally not planning on migrating to the new rest api, as the current one works good enough for me and I find it much easier to modify it for my purposes. |
@djaho OK, even though the REST API that you're using is no longer maintained, we would welcome pull requests with the improvements that you've made. A new version of phpList will be released shortly which includes an automatic updater. In future releases that updater will allow installing the new REST API also, which should make it easier for you to set it up and try it out. |
@samtuke I will test my change for some more time and if all will be well, I will make a pull request. Even though this will be my fist Github pull request :) Thanks for the info about the updater. I will definitely check the new rest api. |
Randomly the API fails to authenticate. It returns false and I have no idea what's causing this, but the code fails to add user to the list. Refresh of the page immediately "solves" the problem because then it works, but it's happening quite often and it's not acceptable for my client.
This is my code (from my own class constructor)
Is there any way to get more info about this issue? Is it known issue? Any ideas how to solve it?
The text was updated successfully, but these errors were encountered: