LuisOsv commented on code in PR #6453:
URL: https://github.com/apache/jmeter/pull/6453#discussion_r2096535166
##########
src/core/src/main/java/org/apache/jmeter/util/JsseSSLManager.java:
##########
@@ -236,7 +237,8 @@ private SSLContext createContext() throws
GeneralSecurityException {
// Wrap the defaults in our custom trust manager
TrustManager[] trustmanagers = tmfactory.getTrustManagers();
- for (int i = 0; i < trustmanagers.length; i++) {
+ int trustManagersCount = trustmanagers.length;
+ for (int i = 0; i < trustManagersCount; i++) {
Review Comment:
Same here, are you agree with new version with index?
```
int indexTrustManager = 0;
for (TrustManager trustManager : trustmanagers) {
if (trustManager instanceof X509TrustManager) {
trustmanagers[indexTrustManager] = new
CustomX509TrustManager(
(X509TrustManager) trustManager);
}
indexTrustManager++;
}
```
--
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: dev-unsubscr...@jmeter.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
