[NIFI-8178] Optimize Initialization of SSLContext in Unit Tests

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.13.0
• Component/s: Tools and Build
• Labels: TLS github-actions security
• Assignee: David Handermann
• Reporter: David Handermann
• Created: 27/Jan/21 18:10
• Updated: 16/Jul/21 15:14
• Resolved: 28/Jan/21 00:49

Description

Multiple unit tests use the StandardSSLContextService implementation class in order to support TLS communication.  As a result of this approach, each test method creates and initializes a new instance of SSLContext, which causes poor performance and intermittent timeouts.

Following the changes implemented in NIFI-8171, other unit tests should be modified to initialize an SSLContext in a method annotated with @BeforeClass and provide that SSLContext using a mock of the SSLContextService interface.

Issue Links

• fixes: NIFI-700 GetHTTP, ListenHTTP, and PostHTTP should be migrated to use SSLContextService fully
• links to: GitHub Pull Request #4787