Bug fix 5586 correctly removed the linefeed from sipserver output. However as the server writes to stdout which is by default buffered this means that we cannot guarantee that responses are being returned in a timely fashion. In practice this can be perceived as extremely slow performance by the self-check unit or a cease of functioning altogether (the unit timesout). In practice reverting 5586 fixes this but as 5586 'does the right thing' this is not a good solution. The proposed solution is that autoflush is specifically set on the output filehandle, to ensure the response message is not buffered. Note because of the nature of the problem it is hard to reproduce the bug consistently,
Created attachment 4698 [details] [review] Proposed Patch
I think this is difficult to test. Would it be ok to verify that SIP still works after patch was applied?
Katrin's correct. If your sipserver works fine with your unit then the test is that it dosen't introduce regressions. If you have 3M (or other) self-checks timing out and restarting or failing this because of timeouts this should fix it. NB logging can be deceptive as the log gets flushed, so you see the outgoing message even if the packet has not been transmitted to the unit. It looks like the unit is at fault unless you monitor the line itself.
Created attachment 5415 [details] [review] no regression after applying patch
Comment on attachment 5415 [details] [review] no regression after applying patch tested with v3.04.04
Simple patch, pushed
The fix for this bug was published in the 3.4.5 release. If you were the reporter of this bug, please take time to verify the fix and update the status of this bug report accordingly. If the bug is fixed to your satisfaction, please close this report.
Confirmed works in production on sites where buffered i/o was causing failures