Allow encoder subclasses to customize the message before it is converted to String #40

nemecec · 2019-11-27T10:45:31Z

Fix for issue #39

…e it is converted to String

osiegmar

As written in #39 this might not be enough, and a test case is missing.

…ferent threads

…t package private level

nemecec · 2019-12-26T12:57:52Z

Added a test and sample usage.

src/test/java/de/siegmar/logbackgelf/custom/CustomGelfEncoder.java

osiegmar · 2019-12-26T14:44:55Z

Thanks!

Fix for #39 - Allow encoder subclasses to customize the message befor…

8fcdad5

…e it is converted to String

osiegmar requested changes Dec 15, 2019

View reviewed changes

Neeme Praks added 11 commits December 26, 2019 13:33

Add test and a sample implementation of the pattern proposed in #39

0bb626b

Make test work in different environments with logs generated from dif…

068b99d

…ferent threads

Fix for coding standards - Use explicit scoping instead of the defaul…

6f7e4fb

…t package private level

Merge branch 'master' into encoder-customization

6b14de1

Add license header

7d48390

Fix indentation

286e71d

Re-organize imports

46d5cfb

Re-organize imports and declare variables final

16098ed

Remove some unnecessary whitespace

ab9aca9

Replace Guava-based hash with standard JDK library

9b30c1c

Fix code style issues

f6a16ef

nemecec requested a review from osiegmar December 26, 2019 12:57

osiegmar requested changes Dec 26, 2019

View reviewed changes

src/test/java/de/siegmar/logbackgelf/custom/CustomGelfEncoder.java Outdated Show resolved Hide resolved

Replace standard JDK library hashing with Guava-based

ae20792

osiegmar merged commit 4d0204e into osiegmar:master Dec 26, 2019

nemecec deleted the encoder-customization branch December 26, 2019 15:04

Provide feedback