Skip to content

Commit

Permalink
Add timeout to TelemetryCollector. Also make it async (although someh…
Browse files Browse the repository at this point in the history
…ow it blocks for me anyways)
  • Loading branch information
maxanier committed Dec 5, 2024
1 parent 2076ef1 commit d7ab68e
Showing 1 changed file with 3 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,24 +3,20 @@
import de.teamlapen.vampirism.REFERENCE;
import de.teamlapen.vampirism.config.VampirismConfig;
import net.minecraft.DetectedVersion;
import net.minecraftforge.srgutils.MinecraftVersion;
import net.neoforged.fml.LogicalSide;
import net.neoforged.fml.ModList;
import net.neoforged.fml.VersionChecker;
import net.neoforged.fml.util.thread.EffectiveSide;
import net.neoforged.neoforge.common.NeoForge;
import net.neoforged.neoforge.internal.versions.neoforge.NeoForgeVersion;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;

import java.io.IOException;
import java.net.URI;
import java.net.URISyntaxException;
import java.net.URLEncoder;
import java.net.http.HttpClient;
import java.net.http.HttpRequest;
import java.net.http.HttpResponse;
import java.nio.charset.StandardCharsets;
import java.time.Duration;
import java.util.HashMap;
import java.util.Map;
import java.util.stream.Collectors;
Expand Down Expand Up @@ -51,8 +47,8 @@ private static void send() {
builder.append("?");
builder.append(params.entrySet().stream().map(s -> s.getKey() + "=" + URLEncoder.encode(s.getValue(), StandardCharsets.UTF_8)).collect(Collectors.joining("&")));

http.send(HttpRequest.newBuilder().uri(new URI(builder.toString())).build(), HttpResponse.BodyHandlers.ofString());
} catch (URISyntaxException | IOException | InterruptedException e) {
http.sendAsync(HttpRequest.newBuilder().uri(new URI(builder.toString())).timeout(Duration.ofSeconds(5)).build(), HttpResponse.BodyHandlers.ofString());
} catch (URISyntaxException e) {
LOGGER.error("Failed to send telemetry data", e);
}
}
Expand Down

0 comments on commit d7ab68e

Please sign in to comment.