Skip to content

Commit 2a2366a

Browse files
Copilotbrunoborges
andcommitted
Fix SpotBugs DM_DEFAULT_ENCODING: Add UTF-8 charset to InputStreamReader
Co-authored-by: brunoborges <129743+brunoborges@users.noreply.github.com>
1 parent 69e9e39 commit 2a2366a

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

src/main/java/com/github/copilot/sdk/CopilotClient.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import java.io.InputStreamReader;
1111
import java.net.Socket;
1212
import java.net.URI;
13+
import java.nio.charset.StandardCharsets;
1314
import java.util.ArrayList;
1415
import java.util.Arrays;
1516
import java.util.HashMap;
@@ -1014,7 +1015,8 @@ private ProcessInfo startCliServer() throws IOException, InterruptedException {
10141015

10151016
// Forward stderr to logger in background
10161017
Thread stderrThread = new Thread(() -> {
1017-
try (BufferedReader reader = new BufferedReader(new InputStreamReader(process.getErrorStream()))) {
1018+
try (BufferedReader reader = new BufferedReader(
1019+
new InputStreamReader(process.getErrorStream(), StandardCharsets.UTF_8))) {
10181020
String line;
10191021
while ((line = reader.readLine()) != null) {
10201022
LOG.fine("[CLI] " + line);
@@ -1029,7 +1031,8 @@ private ProcessInfo startCliServer() throws IOException, InterruptedException {
10291031
Integer detectedPort = null;
10301032
if (!options.isUseStdio()) {
10311033
// Wait for port announcement
1032-
BufferedReader reader = new BufferedReader(new InputStreamReader(process.getInputStream()));
1034+
BufferedReader reader = new BufferedReader(
1035+
new InputStreamReader(process.getInputStream(), StandardCharsets.UTF_8));
10331036
Pattern portPattern = Pattern.compile("listening on port (\\d+)", Pattern.CASE_INSENSITIVE);
10341037
long deadline = System.currentTimeMillis() + 30000;
10351038

0 commit comments

Comments
 (0)