Commit Graph

1037 Commits

Author SHA1 Message Date
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
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
Paul Seiffert f006c2ac47
changelog: bump to v1.0.3 2017-08-15 22:20:55 +02:00
Paul Seiffert e98d2a2169 Merge pull request #616 from rafecolton/make-examples-testable
Replace example files with testable examples
2017-08-15 22:19:09 +02:00
Rafe Colton e3e7388b95 Replace example files with testable examples 2017-08-10 10:46:39 -04:00
Ross McDonald 3bd397e07f Add Telegram hook to README.md. 2017-08-07 16:36:23 -05:00
Dave Clendenan e3d17767d1 MD formatting 2017-08-02 17:35:14 -07:00
Dave Clendenan 9ce1c9e3b5 add github path to log message in readme 2017-08-02 17:33:13 -07:00
Dave Clendenan b1db1b9c67 regex assertion rather than literal, for github path 2017-08-02 17:28:13 -07:00
Dave Clendenan 3cb9e18ef9 test updates
- add tests for callers accessed directly or via function pointer
  (results are unchanged)
- undo unwanted capitalization/export in previous commit
2017-08-02 17:21:18 -07:00
Dave Clendenan 7d48cb786e Merge branch 'master' of https://github.com/sirupsen/logrus
Conflicts:
	alt_exit_test.go
	formatter.go
	json_formatter.go
2017-08-02 13:18:39 -07:00
Damien Mathieu 181d419aa9 Merge pull request #604 from sirupsen/use_ssh_terminal
Remove os-specific IsTerminal methods
2017-07-28 09:42:14 +02:00
dmathieu e66f22976f remove os-specific IsTerminal methods, and use x/crypto/ssh/terminal instead 2017-07-27 15:26:52 +02:00
Damien Mathieu 9aa7601a11 Merge pull request #605 from sirupsen/fix_tempfile_all_platforms
Fix creating temp files on non-unix platforms
2017-07-27 15:26:26 +02:00
dmathieu 60f3438580 fix creating temp files on non-unix platforms 2017-07-27 14:46:09 +02:00
dmathieu d4ae98b177 improve test failure messages for alt_exit_test 2017-07-27 14:06:49 +02:00
Damien Mathieu c2f40cf579 Merge pull request #600 from AndreKR/appveyor
Add appveyor.yml
2017-07-27 09:51:31 +02:00
André Hänsel 6f38f401f7 Add appveyor.yml 2017-07-27 05:09:24 +02:00