-
Notifications
You must be signed in to change notification settings - Fork 9
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
Notify on logout #122
Notify on logout #122
Conversation
maven { | ||
url 'https://maven.google.com/' | ||
name 'Google' | ||
} | ||
} | ||
dependencies { | ||
classpath 'com.android.tools.build:gradle:3.1.2' | ||
classpath "com.android.tools.build:gradle:3.1.1" |
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.
retrieve back the 3.1.2 v
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.
can you check that v 3.1.2 works for you? It did not work for me and for Vitali. The continuous build also failed.
|
||
private static final String OAUTH_ACTIVITY_LOGGING_PATH = "/activity_logging"; | ||
private static final String ID_KEY = "id_token"; | ||
private static final String ACTIVITY_KEY = "eventName"; |
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.
can we name it
EVENT_NAME and
ID_TOKEN ?
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.
changed to EVENT_NAME_KEY, ID_TOKEN_KEY
|
||
ResponseListener resListener = new ResponseListener() { | ||
@Override | ||
public void onSuccess(Response response) { |
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.
add info log on success
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.
already done
} | ||
@Override | ||
public void onFailure(Response response, Throwable t, JSONObject extendedInfo) { | ||
logger.debug("Failed to submit logging request"); |
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.
should it be debug ? why not error?
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.
fixed
No description provided.