quantranhong1999 commented on PR #1824: URL: https://github.com/apache/james-project/pull/1824#issuecomment-1869434235
@thanhbv200585 I helped you setting up the CrowdSec integration test and clean some code [here](https://github.com/quantranhong1999/james-project/commits/thanh-crowdsec-imap/). Please have a look and finish the remaining proxy test case. Context of the commit: - Better handle Crowdsec timeout: wrong CrowdSec endpoint configuration could lead to hanging CrowdSec connection check - Better log for refused connection from an IP - Better reuse `CrowdsecHttpClient` object in `CrowdsecImapConnectionCheck`. Do not create a new client for every check. - Stay away from evil regression `remoteAddress.getHostName` - Only perform connection check if needed. Most of the cases it would be empty connection check therefore no need Flux manipulation. - Set up Integration test for CrowdSec: Plug James log file into CrowdSec, then let CrowdSec and James do their work. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
