commit 1728f5f3e6c316b51bde82405d912189b24edc80
author Mun Yong Jang <myjang@google.com> Mon Nov 27 13:29:08 2017 -0800
committer Commit Bot <commit-bot@chromium.org> Mon Nov 27 21:35:50 2017 +0000
tree 5e18b7296837ff35bfb7dce9ce241d437d99ac1f
parent ad3e639bae26622548fc4df1ef34e2ec08ebedb3

[depot_tools] Fix timestamp issues related to local auth

Bug: 509672
Change-Id: I5abb813d5f78291b4bf2d8e856c6fd71fcad5b68
Reviewed-on: https://chromium-review.googlesource.com/791472
Commit-Queue: Mun Yong Jang <myjang@google.com>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>

auth.py

diff --git a/auth.py b/auth.py
index 11a2113..c415610 100644
--- a/auth.py
+++ b/auth.py
@@ -200,11 +200,17 @@
   if expiry:
     try:
       expiry_dt = datetime.datetime.utcfromtimestamp(expiry)
+      logging.debug(
+        'local_auth: got an access token for '
+        'account "%s" that expires in %d sec',
+        account_id, (expiry_dt - now).total_seconds())
     except (TypeError, ValueError):
       authErr('Invalid expiry in returned token')
-  logging.debug(
-      'local_auth: got an access token for account "%s" that expires in %d sec',
-      account_id, expiry - time.mktime(now.timetuple()))
+  else:
+    logging.debug(
+        'local auth: got an access token for '
+        'account "%s" that does not expire',
+        account_id)
   access_token = AccessToken(access_token, expiry_dt)
   if _needs_refresh(access_token, now=now):
     authErr('local_auth: the returned access token needs to be refreshed')