dzolo commented on pull request #92: URL: https://github.com/apache/commons-net/pull/92#issuecomment-965739581
@garydgregory From my point of view this is a rare but very dangerous bug. If it happens, it is hard to debug the source of the problem (took me 2 days in our Android application). On the other hand, I understand that the backward compatibility is important. I could add an attribute (e.g. `boolean enableResponseCheck = false;`), which can enable this on demand. Than maybe in future (4 major release?) it will be enabled by default. What do you think? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
