Commit Graph

1153 Commits

Author SHA1 Message Date
Lyra Naeseth efab7f37b7 Have prefixFieldClashes respect the JSON FieldMap
Currently, it will fix clashes that don't exist and miss clashes that
do exist due to field remapping.

Fixes #708
2018-02-13 17:49:16 -08:00
Simon Eskildsen 8c0189d9f6
Merge pull request #707 from imjching/logger-with-hooks-race
Fix race condition in entry fireHooks() method
2018-02-13 09:31:10 -05:00
Jay Ching Lim be569094e9
Make fireHooks() method receive a copy of Entry structure to avoid race conditions
Signed-off-by: Jay Ching Lim <imjching@users.noreply.github.com>
2018-02-12 17:26:48 -05:00
Stephen Day 9f91ab2ef9
Merge pull request #704 from phillipj/patch-1
Fix typo in README.md
2018-02-07 10:27:17 -08:00
Stephen Day dcf2d47dfb
Merge pull request #697 from jjcollinge/patch-1
Add Application Insights hook to README
2018-02-06 17:40:45 -08:00
Phillip Johnsen 178041e53c
Fix typo in README.md 2018-02-05 21:59:23 +01:00
Michael Haines 828a649ef2 rename fieldLogger to entry 2018-02-05 12:52:11 -07:00
Michael Haines eeb653535c Lock mutex before formatting to avoid race 2018-02-05 12:44:11 -07:00
Michael Haines efbfdb5f09 Add failing test for using a FieldLogger with hooks inside goroutines 2018-02-05 12:42:00 -07:00
Joni Collinge 0cf9f0bff9
Made text consistent with other hooks 2018-01-30 16:00:33 +00:00
Joni Collinge 516f6c178d
Add Application Insights hook to README
Adds README link to Application Insights hook
2018-01-30 15:58:49 +00:00
Antoine Grondin 768a92a026
Merge pull request #695 from mauricio/master
Fix deadlock on panics at Entry.log
2018-01-29 20:18:52 +02:00
Tony Lee 269eab0f22
Merge branch 'master' into master 2018-01-23 23:04:29 -05:00
Maurício Linhares 977e03308a
Fix deadlock on panics at Entry.log
When calling Entry.log a panic inside some of the
locking blocks could cause the whole logger to deadlock.

One of the ways this could happen is for a hook to cause
a panic, when this happens the lock is never unlocked and
the library deadlocks, causing the code that is calling
it to deadlock as well.

This changes how locking happens with unlocks at defer
blocks so even if a panic happens somewhere along the log
call the library will still unlock and continue to function.
2018-01-22 10:52:46 -05:00
Dennis de Reus 92aece568b TextFormatter behaviour aligned with stdlib log (fixes #167)
stdlib `log` adds a newline at the end of a message if none is present,
otherwise does not. Before this change logrus would always add
a newline, resulting in inconsistent behaviour if stdlib log was
replaced with logrus, and a user would e.g. use 'log.printf("test\n")'
2017-12-29 20:26:35 +01:00
conor eb156905d7 remove .gitignore changes and update AddHook 2017-12-21 14:16:49 -05:00
conor 20cc8e2bc3 remove .gitignore changes 2017-12-21 14:10:48 -05:00
conor 0c03a05a0e mirror and wrap Logger instance methods in exported.go 2017-12-21 14:04:49 -05:00
Simon Eskildsen d682213848 changelog: 1.0.4 2017-12-05 15:32:29 -05:00
Simon Eskildsen 49f0a85ee5
Merge pull request #635 from ernestoalejo/master
Split terminal check to add build tags to support App Engine.
2017-12-05 15:30:37 -05:00
Marianne Feng 1917d221a6 Merge branch 'master' of github.com:mariannefeng/logrus 2017-11-30 23:11:56 -08:00
Neil Isaac b9eceae8f6
fix example 2017-11-21 22:56:37 -05:00
Neil Isaac bf1fb70b2b Add FieldMap support to TestFormatter 2017-11-21 22:43:47 -05:00
Simon Eskildsen 95cd2b9c79
Merge pull request #644 from marccarre/add-promrus-hook
Add promrus to list of hooks.
2017-11-18 07:42:23 -05:00
Simon Eskildsen cb6f9634ca
Merge pull request #658 from xentek/patch-1
Adds `logbeat` hook to README
2017-11-18 07:41:52 -05:00
Simon Eskildsen 5c6f722619
Merge pull request #661 from klautcomputing/patch-1
Fixes a typo in README.md
2017-11-18 07:41:40 -05:00
Simon Eskildsen 4dd868ba91
Merge pull request #669 from StevenYCChou/patch-1
Fix typo in README.md
2017-11-18 07:40:44 -05:00
Yen-Cheng Chou 73a1342386
Fix typo in README.md 2017-11-14 16:58:00 -05:00
Marianne Feng 10d6a5b427 removed useless line from readme 2017-11-06 16:28:37 -08:00
Marianne Feng 639325f81a added pretty print option for json logs 2017-11-06 16:19:47 -08:00
Felix Glaser 9700beb9b6
Update README.md 2017-11-01 14:11:54 -04:00
Eric Marden 1858a8574d Adds `logbeat` hook to README
Adds link to new hook for sending logrus entries to Opbeat service.
2017-10-26 13:58:09 -05:00
Aditya Mukerjee c44d524628 Fix typo in docstring 2017-10-09 11:18:43 -04:00
Marc CARRE 4844e5856d Add promrus to list of hooks. 2017-10-04 15:51:55 +01:00
Ernesto Alejo 7d3ddc68a3 Split terminal check to add build tags to support App Engine. 2017-09-06 19:34:58 +02:00
Albert Nigmatzianov 6137e6b13d Fix typo in docs for New() 2017-09-02 15:05:25 +05:00
Albert Nigmatzianov e59e5eaa92 Fix typo in docs for New() 2017-09-02 15:04:38 +05:00
Albert Nigmatzianov 1fb8c53680 Fix typo in docs for New() 2017-09-02 15:03:37 +05:00
Dave Clendenan 40f571805d Merge branch 'master' of https://github.com/dclendenan/logrus 2017-08-30 15:43:36 -07:00
Dave Clendenan eab1019f63 Merge branch 'master' of https://github.com/sirupsen/logrus 2017-08-30 15:39:09 -07:00
Damien Mathieu 89742aefa4 Merge pull request #612 from ChimeraCoder/add-hook-race
Fix data race for hooks
2017-08-22 15:27:46 +02:00
Damien Mathieu 84573d5f03 Merge pull request #627 from kpfaulkner/azuretablehookref
Added reference to AzureTableHook
2017-08-21 09:31:01 +02:00
Ken Faulkner cd1114dc25 Added reference to AzureTableHook 2017-08-20 12:47:38 +10:00
Aditya Mukerjee 9bc52e3981 Fix test assertion 2017-08-17 15:22:06 +01:00
Aditya Mukerjee c830992a61 Take lock on mutex when firing hooks 2017-08-17 15:22:06 +01:00
Aditya Mukerjee 66230b2871 Add test for race condition in hooks 2017-08-17 15:22:06 +01:00
Aditya Mukerjee 3d1341ce2c Add AddHook method for logger 2017-08-17 15:22:06 +01:00
Damien Mathieu 68806b4b77 Merge pull request #613 from rossmcdonald/rossmcdonald-add-telegram-to-readme
Add Telegram hook to README.md
2017-08-17 10:55:56 +02:00
Damien Mathieu 68e63515d5 Merge pull request #621 from tracer0tong/master
Update README.md to fix link to Kafka hook
2017-08-17 10:41:08 +02:00
Tracer Tong 5efed00cb0 Update README.md to fix link to Kafka hook
Current link is broken. I have made a mix from couple of dead projects and I think I will support my version for long time.
2017-08-17 16:33:45 +09:00