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

Updated LogglyClient.java for tag support #1

Closed
wants to merge 7 commits into from

Fixed Whitespace issues

  • Loading branch information
varshneyjayant committed Aug 12, 2015
commit 6280cbfcd0e0b854249ed27f1b76c407ecf52f56
@@ -135,7 +135,7 @@ public void log(String message, final Callback callback) {
loggly.log(token,
tags,
new TypedString(message),
new retrofit.Callback < LogglyResponse > () {
new retrofit.Callback<LogglyResponse> () {
public void success(LogglyResponse logglyResponse, Response response) {
callback.success();
}
@@ -144,7 +144,7 @@ public void failure(RetrofitError retrofitError) {
callback.failure(retrofitError.getMessage());
}
});
}
}

/**
* Posts several log messages in bulk to Loggly
@@ -191,21 +191,20 @@ public void logBulk(Collection<String> messages, final Callback callback) {
loggly.logBulk(token,
tags,
new TypedString(parcel),
new retrofit.Callback < LogglyResponse > () {
public void success(LogglyResponse logglyResponse,
Response response) {
new retrofit.Callback<LogglyResponse> () {
public void success(LogglyResponse logglyResponse,Response response) {
callback.success();
}

public void failure(RetrofitError retrofitError) {
callback.failure(retrofitError.getMessage());
}
});
}
}

/**
* Combines a collection of messages to be sent to Loggly.
* In order to preserve event boundaries, the new lines in
* In order to preserve event boundaries, the new lines in
* each message are replaced with '\r', which get stripped
* by Loggly.
* @param messages messages to be combined
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.