Commit Graph

755 Commits

Author SHA1 Message Date
earlzo 1893e9a3ed Fixed: comment 2018-02-20 16:28:12 +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
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
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
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
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
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