Skip to content

#255 Asynchronous Sending for RemoteSyslogAppender in log4net #258

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

Merged
merged 2 commits into from
Jun 19, 2025

Conversation

FreeAndNil
Copy link
Contributor

@FreeAndNil FreeAndNil commented Jun 18, 2025

@FreeAndNil FreeAndNil force-pushed the Feature/AsyncRemoteSyslogAppender branch from 1405f9f to e8d7eca Compare June 18, 2025 19:26
@FreeAndNil
Copy link
Contributor Author

@yogitasingh001 I've adjusted your code and added a unit test.
Can you check, whether this version works for your scenario?

@FreeAndNil FreeAndNil requested a review from fluffynuts June 19, 2025 10:50
@FreeAndNil FreeAndNil added this to the 3.1.1 milestone Jun 19, 2025
@yogitasingh001
Copy link
Contributor

Ok sure, thanks for the updates

Copy link
Contributor

@fluffynuts fluffynuts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@FreeAndNil FreeAndNil merged commit 33d668d into master Jun 19, 2025
3 checks passed
@FreeAndNil FreeAndNil deleted the Feature/AsyncRemoteSyslogAppender branch June 19, 2025 13:06
@yogitasingh001
Copy link
Contributor

@FreeAndNil I have verified the changes, working fine. Thanks.

@yogitasingh001
Copy link
Contributor

@FreeAndNil can you please tell, when we can expect the new version with these changes integrated?

@FreeAndNil
Copy link
Contributor Author

Current plan is 3.2.0.preview.1 next week.
I would like to have #257 included.
After that we will release 3.2.0 - due to the holidays probably mid August.

@yogitasingh001
Copy link
Contributor

okay thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants