Commit Graph

747 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
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
Davide D'Agostino 31e110ccae IsTerminal should work in nacl too
I'm running logrus on a nacl environment and I get:

```
github.com/sirupsen/logrus/text_formatter.go:70: undefined: IsTerminal
```
2017-07-25 14:20:14 -07:00
Damien Mathieu 8d4f6a97d3 Merge pull request #595 from mozillazg/patch-1
readme: remove duplicate godoc link
2017-07-25 17:02:49 +02:00
Huang Huang a663abbf13 readme: remove duplicate godoc link 2017-07-25 22:52:40 +08:00
Damien Mathieu 86bd21e371 Merge pull request #593 from sirupsen/buffer_length
Use buffer length to avoid generating strings every time
2017-07-25 10:20:43 +02:00
dmathieu 211aba39c8 use buffer length to avoid generating strings every time 2017-07-25 09:39:35 +02:00
Damien Mathieu 3eef8ce63d Merge pull request #591 from sirupsen/fix_trailing_space
Fix trailing space
2017-07-22 13:39:13 +02:00
dmathieu 159e991025 only add a space between text entries if there is already content
So we don't have a trailing space at the end of each log line
2017-07-21 16:14:28 +02:00
dmathieu b264ba77c3 add test in text formatter checking the formatting 2017-07-21 16:06:23 +02:00