Skip to content
This repository has been archived by the owner on Aug 17, 2020. It is now read-only.

(Problems) Reset AuthToken after 30mins doesn't work #645

Closed
jakubsuchybio opened this issue Aug 12, 2016 · 0 comments · Fixed by #691
Closed

(Problems) Reset AuthToken after 30mins doesn't work #645

jakubsuchybio opened this issue Aug 12, 2016 · 0 comments · Fixed by #691
Labels

Comments

@jakubsuchybio
Copy link
Collaborator

jakubsuchybio commented Aug 12, 2016

Recap:
With help of @ST-Apps we found 2 problems:

  1. After deployment if you don't login and it uses previous authtoken, then after 30mins when the HandleApiFailure triggers it doesn't have proper login settings saved and it tries to DoPtcLogin with null username and password
  2. After deployment if you do login manually, there is bigger problem. After 30 minutes it hits HandleApiFailure, but when it tries to login with correct username and password, Google.Protobuf.dll gives ArgumentNullException.
    It is happening in Login.cs:41 at:
Client.AuthToken = await login.GetAccessToken().ConfigureAwait(false);

login has correct username and password and after F10 to step through it, it gives:

@ST-Apps tried me to create breakpoint at App.xaml.cs:59 if it gets hit, but it doesn't

@ST-Apps ST-Apps added the bug label Aug 13, 2016
@ST-Apps ST-Apps added this to the PoGo-UWP 1.1.0 milestone Aug 13, 2016
ST-Apps added a commit that referenced this issue Aug 14, 2016
Fix #645; Enhance exception handling and async update timer;
00raq00 pushed a commit to 00raq00/PoGo-UWP that referenced this issue Aug 14, 2016
Fix ST-Apps#645 by Logouting, auto-loging in again and completely reinicializing GameMapPage.
Add Exception message propagation from App_UnhandledException.
Change void to Task at ToggleUpdateTimer and fix all usages to await.
Add link to GameMapPageViewModel into GameClient -> needs refactoring.
Temporary fix for compilation error at PlayerProfilePage.xaml.cs:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants