[issue12168] SysLogHandler incorrectly appends \000 to messages

Change by Eryk Sun : -- Removed message: https://bugs.python.org/msg405704 ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue12168] SysLogHandler incorrectly appends \000 to messages

Change by Eryk Sun : -- components: +Library (Lib) -Regular Expressions nosy: -ahmedsayeed1982, ezio.melotti, mrabarnett versions: -Python 3.10 ___ Python tracker ___

[issue12168] SysLogHandler incorrectly appends \000 to messages

Ahmed Sayeed added the comment: This patch fixes a segfault seen when attaching to a process on Solaris. The steps leading to the segfault are: http://www.compilatori.com/tech/xiaomi/ - procfs_target::attach calls do_attach, at this point the inferior's process slot in the

[issue12168] SysLogHandler incorrectly appends \000 to messages

Éric Araujo mer...@netwok.org added the comment: Should this new attribute be documented? -- nosy: +eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12168 ___

[issue12168] SysLogHandler incorrectly appends \000 to messages

Roundup Robot devnull@devnull added the comment: New changeset 3cf5d61fd6d7 by Vinay Sajip in branch '3.2': Documented change for Issue #12168. http://hg.python.org/cpython/rev/3cf5d61fd6d7 New changeset 6658b9b9f5f3 by Vinay Sajip in branch 'default': Merged documentation change for Issue

[issue12168] SysLogHandler incorrectly appends \000 to messages

Roundup Robot devnull@devnull added the comment: New changeset 260b84851d1f by Vinay Sajip in branch '3.2': Issue #12168: SysLogHandler now allows NUL termination to be controlled using a new 'append_nul' attribute on the handler. http://hg.python.org/cpython/rev/260b84851d1f New changeset

[issue12168] SysLogHandler incorrectly appends \000 to messages

Changes by Vinay Sajip vinay_sa...@yahoo.co.uk: -- resolution: - fixed status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12168 ___

[issue12168] SysLogHandler incorrectly appends \000 to messages

Carl Crowder carl.crow...@gmail.com added the comment: Flume certainly could avoid parsing certain values. However, while a syslog application should avoid octet values below 32, they are still legal [1]. I don't think that adjusting flume to reject legal values due to legacy behaviour in

[issue12168] SysLogHandler incorrectly appends \000 to messages

R. David Murray rdmur...@bitdance.com added the comment: And it goes on to say: [...] but a syslog application MAY modify these characters upon reception. For example, it might change them into an escape sequence (e.g., value 0 may be changed to \0) Flume should not break if it receives

[issue12168] SysLogHandler incorrectly appends \000 to messages

Carl Crowder carl.crow...@gmail.com added the comment: Oh, I understand. Flume doesn't break, it handles the \0 just fine, the problem is that I ended up with a message with that additional byte on the end. Sorry for the confusion! -- ___ Python

[issue12168] SysLogHandler incorrectly appends \000 to messages

Petri Lehtinen pe...@digip.org added the comment: FWIW, I once stumbled on this problem, and solved it by making my own log handler that uses functions from the syslog module. -- nosy: +petri.lehtinen ___ Python tracker rep...@bugs.python.org

[issue12168] SysLogHandler incorrectly appends \000 to messages

Vinay Sajip vinay_sa...@yahoo.co.uk added the comment: Petri Lehtinen pe...@digip.org added the comment: FWIW, I once stumbled on this problem, and solved it by making my own log handler that uses functions from the syslog module. @Petri: It's great that it worked for you, but IIRC the

[issue12168] SysLogHandler incorrectly appends \000 to messages

Petri Lehtinen pe...@digip.org added the comment: Vinay Sajip wrote: @Petri: It's great that it worked for you, but IIRC the syslog module has had thread-safety issues at least on some platforms, which is why it's not used in the SysLogHandler implementation (disclaimer: this was a while

[issue12168] SysLogHandler incorrectly appends \000 to messages

Vinay Sajip vinay_sa...@yahoo.co.uk added the comment: Please note: this change is too late for inclusion in Python 3.2.1 :-( as it has already rc status. So I will probably push this change once 3.2.1 is out. -- ___ Python tracker