From f4cb18d102fdb2b43fa003a63140a6bf362ebe6c Mon Sep 17 00:00:00 2001
From: Kavin <20838718+FireMasterK@users.noreply.github.com>
Date: Sun, 30 Oct 2022 23:53:36 +0000
Subject: [PATCH] Add videoId scope and fix sentry transaction trace.

---
 .../me/kavin/piped/server/handlers/StreamHandlers.java | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/main/java/me/kavin/piped/server/handlers/StreamHandlers.java b/src/main/java/me/kavin/piped/server/handlers/StreamHandlers.java
index 569172f..2c3e23c 100644
--- a/src/main/java/me/kavin/piped/server/handlers/StreamHandlers.java
+++ b/src/main/java/me/kavin/piped/server/handlers/StreamHandlers.java
@@ -37,12 +37,18 @@ import static org.schabi.newpipe.extractor.services.youtube.YoutubeParsingHelper
 public class StreamHandlers {
     public static byte[] streamsResponse(String videoId) throws Exception {
 
+        Sentry.configureScope(scope -> {
+            scope.setContexts("videoId", videoId);
+        });
+
         final var futureStream = Multithreading.supplyAsync(() -> {
             ITransaction transaction = Sentry.startTransaction("StreamInfo fetch", "fetch");
             try {
                 return StreamInfo.getInfo("https://www.youtube.com/watch?v=" + videoId);
             } catch (Exception e) {
                 ExceptionUtils.rethrow(e);
+            } finally {
+                transaction.finish();
             }
             return null;
         });
@@ -189,6 +195,10 @@ public class StreamHandlers {
 
     public static byte[] commentsResponse(String videoId) throws Exception {
 
+        Sentry.configureScope(scope -> {
+            scope.setContexts("videoId", videoId);
+        });
+
         CommentsInfo info = CommentsInfo.getInfo("https://www.youtube.com/watch?v=" + videoId);
 
         List<Comment> comments = new ObjectArrayList<>();