Skip to content

Commit f2f4a4f

Browse files
committed
special handling for http error logging
1 parent 7a75467 commit f2f4a4f

4 files changed

Lines changed: 9 additions & 5 deletions

File tree

src/main/java/de/mediathekview/mlib/tool/Log.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,12 @@ public static synchronized void errorLog(int fehlerNummer, String[] text) {
218218
fehlermeldung_(fehlerNummer, null, text);
219219
}
220220

221-
// public static synchronized void systemMeldung(String[] text) {
221+
public static synchronized void logHttpError(String url, int statusCode) {
222+
Exception e = new RuntimeException(String.valueOf(statusCode));
223+
fehlermeldung_(110000000 + statusCode, e, new String[]{ String.format("error %d for URL: %s", statusCode, url)});
224+
}
225+
226+
// public static synchronized void systemMeldung(String[] text) {
222227
// systemmeldung_(text);
223228
// }
224229
public static synchronized void sysLog(String text) {

src/main/java/mServer/crawler/sender/ard/tasks/ArdTaskBase.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ private <A> Optional<A> deserializeUnsafe(final WebTarget target, final Type typ
6262
+ target.getUri().toString()
6363
+ " failed: "
6464
+ response.getStatus();
65-
Log.errorLog(23646387, logText);
65+
Log.logHttpError(target.getUri().toString(), response.getStatus());
6666
LOG.warn(logText);
6767
}
6868
} catch (ProcessingException e) {

src/main/java/mServer/crawler/sender/arte/tasks/ArteVideoInfoTask.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ protected AbstractRecursivConverterTask<ArteVideoInfoDto, TopicUrlDTO> createNew
8383

8484
@Override
8585
protected void handleHttpError(TopicUrlDTO dto, URI url, Response response) {
86-
Log.errorLog(45983789, "http error " + response.getStatus() + ": " + url);
86+
Log.logHttpError(url.toString(), response.getStatus());
8787
log.fatal("A HTTP error {} occurred when getting REST VideoInfo information from: \"{}\".", response.getStatus(), url);
8888
}
8989
}

src/main/java/mServer/crawler/sender/arte/tasks/ArteVideoLinkTask.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88

99
import de.mediathekview.mlib.tool.Log;
1010
import mServer.crawler.sender.MediathekReader;
11-
import mServer.crawler.sender.base.AbstractJsonRestTask;
1211
import mServer.crawler.sender.base.AbstractRecursivConverterTask;
1312
import org.apache.logging.log4j.LogManager;
1413
import org.apache.logging.log4j.Logger;
@@ -59,7 +58,7 @@ protected AbstractRecursivConverterTask<ArteVideoInfoDto, ArteVideoInfoDto> crea
5958

6059
@Override
6160
protected void handleHttpError(ArteVideoInfoDto dto, URI url, Response response) {
62-
Log.errorLog(89723823, "http error: " + response.getStatus() + " " + url);
61+
Log.logHttpError(url.toString(), response.getStatus());
6362
log.fatal("A HTTP error {} occurred when getting REST VideoLink information from: \"{}\".", response.getStatus(), url);
6463
}
6564
}

0 commit comments

Comments
 (0)