<div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Maybe we should switch the defaults to it.<br></blockquote><div><br></div><div>+1 for IDP_PROCESS appender.</div><div><br></div><div>The only downside is that it could increase likelihood of losing log messages in the worst case, e.g. VM crash before dispatch queue is drained. I opted for discardingThreshold=0 in our environment to prevent losing messages when the queue is nearing its capacity, which is the other notable situation where you can lose messages. With those caveats in mind, I felt like it was a win for IDP_PROCESS where we can tolerate losing logs under worst case. In contrast, I avoided the async appender for IDP_AUDIT where we don't want to increase risk of losing data. Also, there's not as much volume in the audit log (ours is 1/15 the size idp-process.log at INFO, much less at DEBUG), so less to gain.</div><div><br></div><div>M<a href="mailto:dev-unsubscribe@shibboleth.net" target="_blank"></a><br>
</div><div><br></div></div></div>