Commit Graph

859 Commits

Author SHA1 Message Date
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
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
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
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
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
Damien Mathieu abee6f9b06 Merge pull request #598 from sirupsen/goling_formatters
Fix golint issues in formatters
2017-07-26 20:39:46 +02:00
Damien Mathieu 259b4b7f45 Merge pull request #594 from sirupsen/sys_unix
Use x/sys/unix instead of syscall
2017-07-26 14:47:05 +02:00
dmathieu 325575f181 fix golint issues in formatters 2017-07-26 14:26:30 +02:00
dmathieu 8a90bf3fff use x/sys for non-unix files 2017-07-26 10:51:31 +02:00
dmathieu f4125cea1b get Termios from x/sys/unix 2017-07-26 10:51:28 +02:00
dmathieu 0af92424f9 install x/sys/unix before running the travis tests 2017-07-26 10:50:52 +02:00
Christy Perez a9ca4bfe68 switch terminal_linux to x/sys/unix from syscall
Closes sirupsen/logrus/issues/515

Signed-off-by: Christy Perez <christy@linux.vnet.ibm.com>
2017-07-26 10:50:51 +02:00
Damien Mathieu 3114d6f617 Merge pull request #597 from DAddYE/master
IsTerminal should work in nacl too
2017-07-26 10:09:27 +02:00