exceptionfactory commented on a change in pull request #4892: URL: https://github.com/apache/nifi/pull/4892#discussion_r593398895
########## File path: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java ########## @@ -1339,26 +1261,7 @@ private Charset getCharsetFromMediaType(MediaType contentType) { * * @return the directory in which the ETag cache should be written */ - private static File getETagCacheDir() { - return Files.createTempDir(); - } - - private static class OverrideHostnameVerifier implements HostnameVerifier { - - private final String trustedHostname; - private final HostnameVerifier delegate; - - private OverrideHostnameVerifier(String trustedHostname, HostnameVerifier delegate) { - this.trustedHostname = trustedHostname; - this.delegate = delegate; - } - - @Override - public boolean verify(String hostname, SSLSession session) { - if (trustedHostname.equalsIgnoreCase(hostname)) { - return true; - } - return delegate.verify(hostname, session); - } + private static File getETagCacheDir() throws IOException { + return Files.createTempDirectory(InvokeHTTP.class.getSimpleName()).toFile(); Review comment: This method is called in the `setUpClient()` method which is annotated with `OnScheduled`. This previous implementation did the same thing using Guava's Files class. The approach seems acceptable since it is called when the OkHttpClient is being configured. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org