CypherpunkSamurai/NewPipeExtractor
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

[YouTube] Do not catch every exception on getErrorMessage

Browse source 
@B0pol suggestion
This commit is contained in:
Stypox 2020-04-11 17:26:31 +02:00
parent 3191bd6c70
commit 68d23defba
No known key found for this signature in database
GPG key ID: 4BDF1B40A49FDD23
1 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeStreamExtractor.java
View file

@ -612,10 +612,11 @@ public class YoutubeStreamExtractor extends StreamExtractor {
@Override @Override
public String getErrorMessage() { public String getErrorMessage() {
try { try {
return getTextFromObject(initialAjaxJson.getObject(2).getObject("playerResponse").getObject("playabilityStatus") return getTextFromObject(initialAjaxJson.getObject(2).getObject("playerResponse")
.getObject("errorScreen").getObject("playerErrorMessageRenderer").getObject("reason")); .getObject("playabilityStatus").getObject("errorScreen")
} catch (Exception e) { .getObject("playerErrorMessageRenderer").getObject("reason"));
return null; } catch (ParsingException | NullPointerException e) {
return null; // no error message
} }
} }