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

[ENHANCEMENT] Make client<>server token refresh so session doesnt timeout #713

Closed
ghost opened this issue Jul 24, 2016 · 9 comments
Closed

Comments

@ghost
Copy link

ghost commented Jul 24, 2016

When I start the bot at my location it is good for about 1 hour, going to PokeStops catching releasing Pokémon but then once that hour or so elapses it just walks round for hours on end (currently 7 hours) doing nothing.

@MikeDX
Copy link
Contributor

MikeDX commented Jul 24, 2016

The session token expires after about an hour (sometimes less, sometimes more). After that it needs to re-login to continue. Your only option at the moment is to restart it.

@ghost
Copy link
Author

ghost commented Jul 24, 2016

Oh, I assumed that it was still running due to the walking notices. Is that not the case then?

@MikeDX
Copy link
Contributor

MikeDX commented Jul 24, 2016

It's still moving around, but it cant perform any tasks as the session is expired (but isnt aware of it)

@gpires
Copy link

gpires commented Jul 24, 2016

@MikeDX Its possible to check if the session is expired?
Then just needs to refresh the token.

@ghost
Copy link
Author

ghost commented Jul 24, 2016

Or just renew it every hour anyway.

@fredrik-hellmangroup fredrik-hellmangroup changed the title How to make the bot perform better? [ENHANCEMENT] Make client<>server token refresh so session doesnt timeout Jul 24, 2016
@MikeDX
Copy link
Contributor

MikeDX commented Jul 24, 2016

@gpires yes, since the PokemonGo-Map repo does exactly this, it gets the session token lifetime at login.

@phupabegim no that is just a dirty hack way of doing it.

@tonz134
Copy link

tonz134 commented Jul 24, 2016

This is my biggest issue right now :<

@MikeDX
Copy link
Contributor

MikeDX commented Jul 24, 2016

Currently working on a fix for this as it is driving me insane :)

@MikeDX
Copy link
Contributor

MikeDX commented Jul 25, 2016

Submitted Pull request #735

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

No branches or pull requests

4 participants