openvidu-server: CDRLoggerWebhook constructor refactoring

pull/581/head
pabloFuente 2020-12-21 15:34:18 +01:00
parent b3670cf893
commit 474e0a70b1
2 changed files with 9 additions and 5 deletions

View File

@ -126,7 +126,8 @@ public class OpenViduServer implements JsonRpcConfigurer {
}
if (openviduConfig.isWebhookEnabled()) {
log.info("OpenVidu Webhook service is enabled");
loggers.add(new CDRLoggerWebhook(openviduConfig));
loggers.add(new CDRLoggerWebhook(openviduConfig.getOpenViduWebhookEndpoint(),
openviduConfig.getOpenViduWebhookHeaders(), openviduConfig.getOpenViduWebhookEvents()));
} else {
log.info("OpenVidu Webhook service is disabled (may be enabled with 'OPENVIDU_WEBHOOK=true')");
}

View File

@ -17,9 +17,13 @@
package io.openvidu.server.webhook;
import java.util.List;
import org.apache.http.Header;
import io.openvidu.server.cdr.CDREvent;
import io.openvidu.server.cdr.CDREventName;
import io.openvidu.server.cdr.CDRLogger;
import io.openvidu.server.config.OpenviduConfig;
import io.openvidu.server.kurento.endpoint.KmsEvent;
import io.openvidu.server.summary.SessionSummary;
@ -27,9 +31,8 @@ public class CDRLoggerWebhook implements CDRLogger {
private HttpWebhookSender webhookSender;
public CDRLoggerWebhook(OpenviduConfig openviduConfig) {
this.webhookSender = new HttpWebhookSender(openviduConfig.getOpenViduWebhookEndpoint(),
openviduConfig.getOpenViduWebhookHeaders(), openviduConfig.getOpenViduWebhookEvents());
public CDRLoggerWebhook(String webhookEndpoint, List<Header> webhookHeaders, List<CDREventName> webhookEvents) {
this.webhookSender = new HttpWebhookSender(webhookEndpoint, webhookHeaders, webhookEvents);
}
@Override