Merge pull request #414 from TeamPiped/sentry-fix

Add videoId scope and fix sentry transaction trace.
This commit is contained in:
Kavin 2022-10-30 23:54:40 +00:00 committed by GitHub
commit 9a143f67b5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -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<>();