mirror of https://github.com/OpenVidu/openvidu.git
openvidu-server: better logs about enabled and disabled services
parent
9f6f89bbad
commit
af31713c66
|
@ -101,11 +101,16 @@ public class OpenViduServer implements JsonRpcConfigurer {
|
|||
public CallDetailRecord cdr(OpenviduConfig openviduConfig) {
|
||||
List<CDRLogger> loggers = new ArrayList<>();
|
||||
if (openviduConfig.isCdrEnabled()) {
|
||||
log.info("OpenVidu CDR is enabled");
|
||||
log.info("OpenVidu CDR service is enabled");
|
||||
loggers.add(new CDRLoggerFile());
|
||||
} else {
|
||||
log.info("OpenVidu CDR service is disabled");
|
||||
}
|
||||
if (openviduConfig.isWebhookEnabled()) {
|
||||
log.info("OpenVidu Webhook service is enabled");
|
||||
loggers.add(new CDRLoggerWebhook(openviduConfig));
|
||||
} else {
|
||||
log.info("OpenVidu Webhook service is disabled");
|
||||
}
|
||||
return new CallDetailRecord(loggers);
|
||||
}
|
||||
|
|
|
@ -440,11 +440,6 @@ public class OpenviduConfig {
|
|||
break;
|
||||
case "openvidu.webhook.endpoint":
|
||||
webhookEndpoint = checkString(parameters, parameter);
|
||||
try {
|
||||
checkWebhookEndpoint(webhookEndpoint);
|
||||
} catch (Exception e) {
|
||||
throw new Exception("Property 'openvidu.webhook.endpoint' is not valid: " + e.getMessage());
|
||||
}
|
||||
break;
|
||||
case "openvidu.streams.video.max-recv-bandwidth":
|
||||
checkIntegerNonNegative(parameters, parameter, admitStringified);
|
||||
|
|
|
@ -121,6 +121,7 @@ public class RecordingManager {
|
|||
@PostConstruct
|
||||
public void init() {
|
||||
if (this.openviduConfig.isRecordingModuleEnabled()) {
|
||||
log.info("OpenVidu recording service is enabled");
|
||||
try {
|
||||
this.initializeRecordingManager();
|
||||
} catch (OpenViduException e) {
|
||||
|
@ -139,6 +140,8 @@ public class RecordingManager {
|
|||
log.error(finalErrorMessage + ". Shutting down OpenVidu Server");
|
||||
System.exit(1);
|
||||
}
|
||||
} else {
|
||||
log.info("OpenVidu recording service is disabled");
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue