-
-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Log4netTextFormatter generates invalid XML #21
Comments
I will take a look at it. I was influenced by the log4net logger, but have to make sure that I'm not doing anything the log4net logger isn't. Thanks for notifying me! |
FantasticFiasco
added a commit
that referenced
this issue
Jan 29, 2018
Add namespace to XML generated by Log4jTextFormatter and Log4netTextFormatter. Closes #21
FantasticFiasco
added a commit
that referenced
this issue
Jan 29, 2018
Add namespace to XML generated by Log4jTextFormatter and Log4netTextFormatter. Closes #21
An official version (v4.0.1) of the package has been uploaded to nuget.org. Sorry it took this long to fix the issue! |
Thanks.
…--
Gian Maria Ricci
Cell: +39 320 0136949
<http://mvp.microsoft.com/en-us/mvp/Gian%20Maria%20Ricci-4025635>
<http://www.linkedin.com/in/gianmariaricci> <https://twitter.com/alkampfer>
<http://feeds.feedburner.com/AlkampferEng>
On Mon, Jan 29, 2018 at 10:03 PM, Mattias Kindborg @FantasticFiasco < ***@***.***> wrote:
An official version (v4.0.1) of the package has been uploaded to nuget.org
.
Sorry it took this long to fix the issue!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#21 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAV4kSOnZLnthxXSGI6gW0EPQx1tDEgDks5tPjIYgaJpZM4QTKGo>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In Log4netTextFormatter class, xml start with
output.Write("<log4net:event");
But actually the namespace log4net is never declared, the XML generated is invalid.
Possible and simple fix is removing the namespace entirely.
The text was updated successfully, but these errors were encountered: