Found a few more places using 'String.getBytes()' - all are working with UTF-8 strings

Author: mcculls

dd-java-agent/agent-crashtracking/src/main/java/datadog/crashtracking/buildid/ElfBuildIdExtractor.java

@@ -4,6 +4,7 @@
 import java.io.RandomAccessFile;
 import java.nio.ByteBuffer;
 import java.nio.ByteOrder;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Path;
 import java.util.Arrays;
 import org.slf4j.Logger;
@@ -31,7 +32,7 @@ public class ElfBuildIdExtractor implements BuildIdExtractor {
 
   // Note header constants
   private static final int NT_GNU_BUILD_ID = 3;
-  private static final byte[] GNU_NOTE_NAME = "GNU\0".getBytes();
+  private static final byte[] GNU_NOTE_NAME = "GNU\0".getBytes(StandardCharsets.UTF_8);
 
   @Override
   public String extractBuildId(Path file) {

dd-java-agent/agent-iast/src/main/java/com/datadog/iast/model/json/VulnerabilityEncoding.java

@@ -5,6 +5,7 @@
 import com.squareup.moshi.Moshi;
 import datadog.trace.api.iast.telemetry.IastMetric;
 import datadog.trace.api.iast.telemetry.IastMetricCollector;
+import java.nio.charset.StandardCharsets;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -25,7 +26,7 @@ public class VulnerabilityEncoding {
   public static String toJson(final VulnerabilityBatch value) {
     try {
       String json = BATCH_ADAPTER.toJson(value);
-      return json.getBytes().length > MAX_SPAN_TAG_SIZE
+      return json.getBytes(StandardCharsets.UTF_8).length > MAX_SPAN_TAG_SIZE
           ? getExceededTagSizeJson(new TruncatedVulnerabilities(value.getVulnerabilities()))
           : json;
     } catch (Exception ex) {

dd-java-agent/src/main/java/datadog/trace/bootstrap/BootstrapInitializationTelemetry.java

@@ -8,6 +8,7 @@
 import de.thetaphi.forbiddenapis.SuppressForbidden;
 import java.io.Closeable;
 import java.io.OutputStream;
+import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.LinkedHashMap;
 import java.util.List;
@@ -319,7 +320,7 @@ public void run() {
 
       // Run forwarder and mute tracing for subprocesses executed in by dd-java-agent.
       try (final Closeable ignored = muteTracing()) {
-        byte[] payload = telemetry.toString().getBytes();
+        byte[] payload = telemetry.toString().getBytes(StandardCharsets.UTF_8);
 
         Process process = builder.start();
         try (OutputStream out = process.getOutputStream()) {

dd-java-agent/src/main/java6/datadog/trace/bootstrap/AgentPreCheck.java

@@ -8,6 +8,7 @@
 import java.io.PrintStream;
 import java.lang.instrument.Instrumentation;
 import java.lang.reflect.Method;
+import java.nio.charset.StandardCharsets;
 
 /** Special lightweight pre-main class that skips installation on incompatible JVMs. */
 public class AgentPreCheck {
@@ -189,7 +190,7 @@ public void run() {
         OutputStream out = null;
         try {
           out = process.getOutputStream();
-          out.write(payload.getBytes());
+          out.write(payload.getBytes(StandardCharsets.UTF_8));
         } finally {
           if (out != null) {
             out.close();

dd-trace-core/src/main/java/datadog/trace/civisibility/writer/ddintake/CiTestCovMapperV2.java

@@ -189,7 +189,7 @@ private static class PayloadV2 extends Payload {
 
     // backend requires _some_ JSON to be present
     private static final RequestBody DUMMY_JSON_BODY =
-        jsonRequestBodyOf("{\"dummy\":true}".getBytes());
+        jsonRequestBodyOf("{\"dummy\":true}".getBytes(StandardCharsets.UTF_8));
 
     private final boolean compressionEnabled;
 

internal-api/src/main/java/datadog/trace/api/datastreams/TransactionInfo.java

@@ -2,6 +2,7 @@
 
 import java.nio.ByteBuffer;
 import java.nio.ByteOrder;
+import java.nio.charset.StandardCharsets;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.atomic.AtomicInteger;
@@ -38,7 +39,7 @@ private int generateCheckpointId(String checkpoint) {
     int id = ID_COUNTER.getAndIncrement();
 
     // update cache bytes
-    byte[] checkpointBytes = checkpoint.getBytes();
+    byte[] checkpointBytes = checkpoint.getBytes(StandardCharsets.UTF_8);
     byte[] bytesToAdd = new byte[checkpointBytes.length + 2];
     bytesToAdd[0] = (byte) id;
     bytesToAdd[1] = (byte) checkpointBytes.length;
@@ -56,7 +57,7 @@ private static synchronized void appendCacheBytes(byte[] bytes) {
   }
 
   public byte[] getBytes() {
-    byte[] idBytes = id.getBytes();
+    byte[] idBytes = id.getBytes(StandardCharsets.UTF_8);
 
     // long ids will be truncated
     int idLen = Math.min(idBytes.length, MAX_ID_SIZE);

internal-api/src/main/java/datadog/trace/api/git/RawParseUtils.java

@@ -6,6 +6,7 @@
 import java.nio.charset.CharacterCodingException;
 import java.nio.charset.CharsetDecoder;
 import java.nio.charset.CodingErrorAction;
+import java.nio.charset.StandardCharsets;
 import java.util.Arrays;
 
 /**
@@ -17,8 +18,8 @@ public final class RawParseUtils {
 
   private RawParseUtils() {}
 
-  public static final byte[] COMMITTER = "committer ".getBytes();
-  public static final byte[] AUTHOR = "author ".getBytes();
+  public static final byte[] COMMITTER = "committer ".getBytes(StandardCharsets.UTF_8);
+  public static final byte[] AUTHOR = "author ".getBytes(StandardCharsets.UTF_8);
 
   private static final byte[] digits10;
 

internal-api/src/main/java/datadog/trace/util/FNV64Hash.java

@@ -1,5 +1,7 @@
 package datadog.trace.util;
 
+import java.nio.charset.StandardCharsets;
+
 /**
  * Calculates the FNV 64 bit hash. Longs should be treated as though they were unsigned
  *
@@ -15,11 +17,11 @@ public enum Version {
   }
 
   public static long generateHash(String data, Version version) {
-    return generateHash(data.getBytes(), version);
+    return generateHash(data.getBytes(StandardCharsets.UTF_8), version);
   }
 
   public static long continueHash(long currentHash, String data, Version version) {
-    return continueHash(currentHash, data.getBytes(), version);
+    return continueHash(currentHash, data.getBytes(StandardCharsets.UTF_8), version);
   }
 
   public static long generateHash(byte[] data, Version version) {