Skip to content
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

Regression in log4net 3.0.3 when using log4net.Ext.json 3.0.2 #211

Closed
Midhun458 opened this issue Nov 25, 2024 · 7 comments
Closed

Regression in log4net 3.0.3 when using log4net.Ext.json 3.0.2 #211

Midhun458 opened this issue Nov 25, 2024 · 7 comments

Comments

@Midhun458
Copy link

Midhun458 commented Nov 25, 2024

Conf ;

<threshold value="WARN"/>
<file type="log4net.Util.PatternString" value="path" />
<appendToFile value="true" />
<layout type="log4net.Layout.SerializedLayout, log4net.Ext.Json">
	<decorator type="log4net.Layout.Decorators.StandardTypesDecorator, log4net.Ext.Json" />
	<default />
	<!--explicit default members-->
	<remove value="ndc" />
	<remove value="message" />
	<!--remove the default preformatted message member-->
	<member value="clientid:clientid" />
	<member value="message:messageobject" />
	<!--add raw message-->
</layout>

Windows .net framework 4.7.2 log4net.Ext.Json version 3.0.1. it suppose to write json to the defined path.

@Midhun458 Midhun458 changed the title The given configuration works with versions 3.0.1 and 3.0.2, writing logs to the specified path. However, it does not work with version 3.0.3 The given configuration works with versions 3.0.1 and 3.0.2, writing json logs to the specified path. However, it does not work with version 3.0.3 Nov 25, 2024
@gdziadkiewicz
Copy link
Contributor

I will take a look

@gdziadkiewicz
Copy link
Contributor

@Midhun458 I reproduced it and confirmed that it started with log4net 3.0.3. Work on the fix is in progress.

@gdziadkiewicz
Copy link
Contributor

Hi @Midhun458 , I implemented a fix in log4net.Ext.Json 3.0.3-develop1560902832 . It resolves the problem on my side; can you test it on yours?

@gdziadkiewicz
Copy link
Contributor

@Midhun458 did you manage to try it out?

@Midhun458
Copy link
Author

Midhun458 commented Dec 3, 2024

image
This does write log without any issue. @gdziadkiewicz

@gdziadkiewicz
Copy link
Contributor

@Midhun458 I released full-fledged 3.0.3 version of log4net.Ext.Json
@FreeAndNil we can close the issue

@FreeAndNil
Copy link
Contributor

@gdziadkiewicz thanks a lot.

@FreeAndNil FreeAndNil changed the title The given configuration works with versions 3.0.1 and 3.0.2, writing json logs to the specified path. However, it does not work with version 3.0.3 Regression in log4net 3.0.3 when using log4net.Ext.json 3.0.2 Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants