diff --git a/openvidu-server/src/main/java/io/openvidu/server/recording/service/RecordingManager.java b/openvidu-server/src/main/java/io/openvidu/server/recording/service/RecordingManager.java index bdff3b2f..5419feb5 100644 --- a/openvidu-server/src/main/java/io/openvidu/server/recording/service/RecordingManager.java +++ b/openvidu-server/src/main/java/io/openvidu/server/recording/service/RecordingManager.java @@ -420,16 +420,19 @@ public class RecordingManager { Recording recording = new Recording(json); if (io.openvidu.java.client.Recording.Status.ready.equals(recording.getStatus()) || io.openvidu.java.client.Recording.Status.failed.equals(recording.getStatus())) { - String recordingUrl = openviduConfig.getFinalUrl() + "recordings/" + recording.getId() + "/" - + recording.getName() + "." + this.getExtensionFromRecording(recording); - recording.setUrl(recordingUrl); + recording.setUrl(getRecordingUrl(recording)); } return recording; } return null; } - public String getExtensionFromRecording(Recording recording) { + public String getRecordingUrl(Recording recording) { + return openviduConfig.getFinalUrl() + "recordings/" + recording.getId() + "/" + + recording.getName() + "." + this.getExtensionFromRecording(recording); + } + + private String getExtensionFromRecording(Recording recording) { if (io.openvidu.java.client.Recording.OutputMode.INDIVIDUAL.equals(recording.getOutputMode())) { return "zip"; } else if (recording.hasVideo()) { diff --git a/openvidu-server/src/main/java/io/openvidu/server/recording/service/RecordingService.java b/openvidu-server/src/main/java/io/openvidu/server/recording/service/RecordingService.java index f0f59056..011f28ed 100644 --- a/openvidu-server/src/main/java/io/openvidu/server/recording/service/RecordingService.java +++ b/openvidu-server/src/main/java/io/openvidu/server/recording/service/RecordingService.java @@ -104,6 +104,9 @@ public abstract class RecordingService { io.openvidu.java.client.Recording.Status status = io.openvidu.java.client.Recording.Status.failed .equals(recording.getStatus()) ? io.openvidu.java.client.Recording.Status.failed : io.openvidu.java.client.Recording.Status.ready; + + // Status is now failed or ready. Url property must be defined + recording.setUrl(recordingManager.getRecordingUrl(recording)); final String entityFile = this.openviduConfig.getOpenViduRecordingPath() + recording.getId() + "/" + RecordingManager.RECORDING_ENTITY_FILE + recording.getId();