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

Log ERROR instead of WARN of TimeToBeReceivedOnForwardedMessages is used #4508

Closed
timbussmann opened this issue Feb 27, 2017 · 6 comments
Closed
Milestone

Comments

@timbussmann
Copy link
Contributor

As the configurationsection will likely be removed completely without obsoletion attributes in v7 (as not supported by .net core), the idea was to increase the log severity with 6.3 to error in order to increase the likelihood of users noticing the log entry.

@danielmarbach
Copy link
Contributor

danielmarbach commented Feb 27, 2017 via email

@WilliamBZA
Copy link
Member

Milestone?

@danielmarbach
Copy link
Contributor

@WilliamBZA yeah that is what we usually do. Was a hidden hint to create the milestone ;)

@WilliamBZA WilliamBZA added this to the 6.3.0 milestone Feb 27, 2017
@timbussmann
Copy link
Contributor Author

weird, I was pretty sure I added the milestone when I created the issue. Thanks @WilliamBZA

@timbussmann
Copy link
Contributor Author

@Particular/nservicebus-maintainers any reason not to extend the scope of this issue to all configuration section warnings (see also #4644 (comment))?

@timbussmann
Copy link
Contributor Author

closing this as #4644 has been merged and will be included in 6.3

@andreasohlund andreasohlund changed the title Switch TimeToBeReceivedOnForwardedMessages log warning to error. Log ERROR instead of WARN of TimeToBeReceivedOnForwardedMessages is used May 18, 2017
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

No branches or pull requests

3 participants