Skip to content

Commit e237ea6

Browse files
committed
Re-apply feedback
1 parent af8dacb commit e237ea6

File tree

1 file changed

+11
-10
lines changed

1 file changed

+11
-10
lines changed

instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/semconv/http/HttpClientAttributesExtractor.java

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -172,9 +172,10 @@ private static String redactQueryParameters(String url) {
172172
}
173173

174174
StringBuilder redactedParameters = new StringBuilder();
175-
boolean paramToRedact = false; // To be able to skip the characters of the parameters to redact
176-
boolean paramNameDetected = false;
177-
boolean reference = false;
175+
boolean inRedactedParamValue =
176+
false; // To be able to skip the characters of the parameters to redact
177+
boolean inParamValue = false;
178+
boolean inReference = false;
178179

179180
// To build a parameter name until we reach the '=' character
180181
// If the parameter name is a one to redact, we will redact the value
@@ -183,25 +184,25 @@ private static String redactQueryParameters(String url) {
183184
for (int i = questionMarkIndex + 1; i < url.length(); i++) {
184185
char currentChar = url.charAt(i);
185186
if (currentChar == '=') {
186-
paramNameDetected = true;
187+
inParamValue = true;
187188
redactedParameters.append(currentParamName);
188189
redactedParameters.append('=');
189190
if (PARAMS_TO_REDACT.contains(currentParamName.toString())) {
190191
redactedParameters.append("REDACTED");
191-
paramToRedact = true;
192+
inRedactedParamValue = true;
192193
}
193194
} else if (currentChar == '&') { // New parameter delimiter
194195
redactedParameters.append('&');
195-
paramNameDetected = false;
196-
paramToRedact = false;
196+
inParamValue = false;
197+
inRedactedParamValue = false;
197198
currentParamName.setLength(
198199
0); // To avoid creating a new StringBuilder for each new parameter
199200
} else if (currentChar == '#') { // Reference delimiter
200-
reference = true;
201+
inReference = true;
201202
redactedParameters.append('#');
202-
} else if (!paramNameDetected) {
203+
} else if (!inParamValue) {
203204
currentParamName.append(currentChar);
204-
} else if (!paramToRedact || reference) {
205+
} else if (!inRedactedParamValue || inReference) {
205206
redactedParameters.append(currentChar);
206207
}
207208
}

0 commit comments

Comments
 (0)