Skip to content

Commit

Permalink
Merge branch 'feature-bbb-integration-v3' into mconf-build
Browse files Browse the repository at this point in the history
  • Loading branch information
fcecagno committed Mar 31, 2022
2 parents 2bac8a5 + fd93899 commit 1068c09
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 0 deletions.
17 changes: 17 additions & 0 deletions back/src/Services/SocketManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ import {
GroupUsersUpdateMessage,
LockGroupPromptMessage,
RoomMessage,
ErrorMessage,
} from "../Messages/generated/messages_pb";
import { User, UserSocket } from "../Model/User";
import { ProtobufUtils } from "../Model/Websocket/ProtobufUtils";
Expand Down Expand Up @@ -622,6 +623,22 @@ export class SocketManager {
const meetingId = joinBBBMeetingMessage.getMeetingid();
const meetingName = joinBBBMeetingMessage.getMeetingname();

if (BBB_URL.length == 0 || BBB_SECRET.length == 0) {
const errorStr =
"Unable to join the conference because either " +
"the BBB_URL or BBB_SECRET environment variables are not set.";

console.error(errorStr);

const errorMessage = new ErrorMessage();
errorMessage.setMessage(errorStr);

const serverToClientMessage = new ServerToClientMessage();
serverToClientMessage.setErrormessage(errorMessage);
user.socket.write(serverToClientMessage);

return;
}
const api = BigbluebuttonJs.api(BBB_URL, BBB_SECRET);
// It seems bbb-api is limiting password length to 50 chars
const maxPWLen = 50;
Expand Down
1 change: 1 addition & 0 deletions front/src/Connexion/RoomConnection.ts
Original file line number Diff line number Diff line change
Expand Up @@ -440,6 +440,7 @@ export class RoomConnection implements RoomConnection {
case "bbbMeetingClientURLMessage": {
this._bbbMeetingClientURLMessageStream.next(message.bbbMeetingClientURLMessage);
break;
}
case "groupUsersUpdateMessage": {
this._groupUsersUpdateMessageStream.next(message.groupUsersUpdateMessage);
break;
Expand Down

0 comments on commit 1068c09

Please sign in to comment.