-
Notifications
You must be signed in to change notification settings - Fork 475
Closes #8492: Gracefully handle network failures while obtaining token server url #8498
Closes #8492: Gracefully handle network failures while obtaining token server url #8498
Conversation
ba7ab49
to
2eedbf0
Compare
2eedbf0
to
133401d
Compare
133401d
to
d484fb2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Curious to learn more / see below. Also, did you mean to commit the metrics files too?
@@ -184,8 +184,10 @@ class FirefoxAccount internal constructor( | |||
} | |||
} | |||
|
|||
override fun getTokenServerEndpointURL(): String { | |||
return inner.getTokenServerEndpointURL() | |||
override suspend fun getTokenServerEndpointURL() = withContext(scope.coroutineContext) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Curious how this didn't fail before? Was it wrapped in an handleFxaExceptions
and we lost that? Or are we calling it more often now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still investigating this. It's neither of those, I'll let you know once I sort this out. Maybe it was failing and we didn't notice, somehow..?
RE metrics - yup, these doc files keep getting out of date, somehow, I commit these diffs when I see them locally. Quite annoying. |
…obtaining token server url
d484fb2
to
a9bcd73
Compare
Pull Request checklist
After merge