This repository was archived by the owner on Dec 14, 2023. It is now read-only.
Adds the ability to send a raw message (useful for json message)#25
Open
jedi4ever wants to merge 2 commits intoLogentriesCommunity:masterfrom
Open
Adds the ability to send a raw message (useful for json message)#25jedi4ever wants to merge 2 commits intoLogentriesCommunity:masterfrom
jedi4ever wants to merge 2 commits intoLogentriesCommunity:masterfrom
Conversation
Contributor
|
closing as merge conflict |
Author
|
euh @udangel-r7 ? would you like me to rebase or implemented this in the new version? |
Contributor
|
As nobody responded to this PR (and it seemed to became stale) I closed it. If rebase it + fix the formatting issues I am happy to merge it. re-opening for the time being |
|
Being able to send raw data (more precisely: custom json) would be much appreciated. Currently our JSON is parsed correctly and added within the 'message' property. While this approach is already a decent benefit it would be nice to just send custom json (without having device-id, timestamp, etc. auto-attached) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
By default le_android adds the timestamp and loglevel to the message you send to logentries.
This PR adds the setSendRawMessage(boolean) function so you can remove that log prefix.
We use it for logging plain json log so parsing works correctly.
The settings is default to false so it is backwards compatible.
P.S. I think it's better to add getter/setters then to provide other constructors to the logger