openvidu-server: logging refactoring

pull/391/head
pabloFuente 2020-02-16 16:27:29 +01:00
parent 1713d79b4a
commit dc8a0449b9
3 changed files with 8 additions and 4 deletions

View File

@ -104,13 +104,13 @@ public class OpenViduServer implements JsonRpcConfigurer {
log.info("OpenVidu CDR service is enabled"); log.info("OpenVidu CDR service is enabled");
loggers.add(new CDRLoggerFile()); loggers.add(new CDRLoggerFile());
} else { } else {
log.info("OpenVidu CDR service is disabled"); log.info("OpenVidu CDR service is disabled (may be enable with 'openvidu.cdr=true')");
} }
if (openviduConfig.isWebhookEnabled()) { if (openviduConfig.isWebhookEnabled()) {
log.info("OpenVidu Webhook service is enabled"); log.info("OpenVidu Webhook service is enabled");
loggers.add(new CDRLoggerWebhook(openviduConfig)); loggers.add(new CDRLoggerWebhook(openviduConfig));
} else { } else {
log.info("OpenVidu Webhook service is disabled"); log.info("OpenVidu Webhook service is disabled (may be enabled with 'openvidu.webhook=true')");
} }
return new CallDetailRecord(loggers); return new CallDetailRecord(loggers);
} }

View File

@ -470,7 +470,11 @@ public abstract class SessionManager {
if (openviduConfig.isRecordingModuleEnabled() && stopRecording if (openviduConfig.isRecordingModuleEnabled() && stopRecording
&& this.recordingManager.sessionIsBeingRecorded(session.getSessionId())) { && this.recordingManager.sessionIsBeingRecorded(session.getSessionId())) {
try {
recordingManager.stopRecording(session, null, RecordingManager.finalReason(reason)); recordingManager.stopRecording(session, null, RecordingManager.finalReason(reason));
} catch (OpenViduException e) {
log.error("Error stopping recording of session {}: {}", session.getSessionId(), e.getMessage());
}
} }
final String mediaNodeId = session.getMediaNodeId(); final String mediaNodeId = session.getMediaNodeId();

View File

@ -186,7 +186,7 @@ public abstract class KmsManager {
// TODO: This should be done here instead of after KurentoClient.create method returns // TODO: This should be done here instead of after KurentoClient.create method returns
// kms.setKurentoClientConnected(true); // kms.setKurentoClientConnected(true);
// kms.setTimeOfKurentoClientConnection(System.currentTimeMillis()); // kms.setTimeOfKurentoClientConnection(System.currentTimeMillis());
log.warn("Kurento Client is now connected to KMS {} with uri {}", kmsId, kms.getUri()); log.info("Kurento Client is now connected to KMS {} with uri {}", kmsId, kms.getUri());
} }
}; };
} }