]> sipb.mit.edu Git - ikiwiki.git/blobdiff - doc/bugs/syslog_fails_with_non-ASCII_wikinames.mdwn
i rebased all my branches to the latest release and renamed some branches, 2 bugs...
[ikiwiki.git] / doc / bugs / syslog_fails_with_non-ASCII_wikinames.mdwn
index 712205b7ac0ec7c41e64929d5139b8f91b9a511d..dad974a81bd51d87f596a21e94833d3eac446d17 100644 (file)
@@ -1,4 +1,4 @@
-[[!template  id=gitbranch branch=anarcat/syslog_utf8 author="[[anarcat]]"]]
+[[!template  id=gitbranch branch=anarcat/dev/syslog_utf8 author="[[anarcat]]"]]
 
 [[this feature|todo/syslog_should_show_wiki_name]] made it so syslog doesn't work anymore if the site being logged has non-ASCII characters it in.
 
@@ -14,8 +14,10 @@ My guess is this sits somewhere here:
 
 Yet I am not sure how to fix that kind of problem in Perl... --[[anarcat]]
 
-If I remove the "eval" above, I get:
-
-    Error: Wide character in syswrite at /usr/lib/perl/5.14/Sys/Syslog.pm line 485.
-
-I have improved a little the error handling in log_message() so that we see *something* when syslog fails, see the branch documented above. I can also confirm that reverting [[todo/syslog_should_show_wiki_name]] fixes the bug. Finally, I have a unit test that reproduces the problem in git, and a working [[!taglink patch]] for the bug, again in git.
+> If I remove the "eval" above, I get:
+> 
+>     Error: Wide character in syswrite at /usr/lib/perl/5.14/Sys/Syslog.pm line 485.
+> 
+> I have improved a little the error handling in log_message() so that we see *something* when syslog fails, see the branch documented above. I can also confirm that  reverting [[todo/syslog_should_show_wiki_name]] fixes the bug. Finally, I have a unit test that reproduces the problem in git, and a working [[!taglink patch]] for the bug, again in git.
+> 
+> > One last note: I noticed that this problem also happens elsewhere in ikiwiki. For example, the [[plugins/notifyemail]] plugin will silently fail to send notifications if the pages contain unicode. The [[plugins/notifychanges]] plugin I am working on (in [[todo/option to send only the diff in notifyemail]]) seems to be working around the issue so far, but there's no telling which similar problem are out there.