diff --git a/akka-bbb-apps/src/test/scala/org/bigbluebutton/core/AppsTestFixtures.scala b/akka-bbb-apps/src/test/scala/org/bigbluebutton/core/AppsTestFixtures.scala
index d82f2d88158949416a1445f52094a838bc455d6e..98311aa9bd945c765302413cd34af5dee7bb7183 100755
--- a/akka-bbb-apps/src/test/scala/org/bigbluebutton/core/AppsTestFixtures.scala
+++ b/akka-bbb-apps/src/test/scala/org/bigbluebutton/core/AppsTestFixtures.scala
@@ -39,6 +39,7 @@ trait AppsTestFixtures {
val maxUsers = 25
val guestPolicy = "ALWAYS_ASK"
val allowModsToUnmuteUsers = false
+ val authenticatedGuest = false
val red5DeskShareIPTestFixture = "127.0.0.1"
val red5DeskShareAppTestFixtures = "red5App"
@@ -58,7 +59,7 @@ trait AppsTestFixtures {
modOnlyMessage = modOnlyMessage)
val voiceProp = VoiceProp(telVoice = voiceConfId, voiceConf = voiceConfId, dialNumber = dialNumber, muteOnStart = muteOnStart)
val usersProp = UsersProp(maxUsers = maxUsers, webcamsOnlyForModerator = webcamsOnlyForModerator,
- guestPolicy = guestPolicy, allowModsToUnmuteUsers = allowModsToUnmuteUsers)
+ guestPolicy = guestPolicy, allowModsToUnmuteUsers = allowModsToUnmuteUsers, authenticatedGuest = authenticatedGuest)
val metadataProp = new MetadataProp(metadata)
val defaultProps = DefaultProps(meetingProp, breakoutProps, durationProps, password, recordProp, welcomeProp, voiceProp,
diff --git a/bbb-common-message/src/main/scala/org/bigbluebutton/common2/domain/Meeting2x.scala b/bbb-common-message/src/main/scala/org/bigbluebutton/common2/domain/Meeting2x.scala
index 287a8c1e6298aa7a4c26a27e1b77de26ba26c8dd..67f842e615e49c18ecbfa4f13134a1a2a978a0f7 100755
--- a/bbb-common-message/src/main/scala/org/bigbluebutton/common2/domain/Meeting2x.scala
+++ b/bbb-common-message/src/main/scala/org/bigbluebutton/common2/domain/Meeting2x.scala
@@ -26,7 +26,7 @@ case class WelcomeProp(welcomeMsgTemplate: String, welcomeMsg: String, modOnlyMe
case class VoiceProp(telVoice: String, voiceConf: String, dialNumber: String, muteOnStart: Boolean)
-case class UsersProp(maxUsers: Int, webcamsOnlyForModerator: Boolean, guestPolicy: String, allowModsToUnmuteUsers: Boolean)
+case class UsersProp(maxUsers: Int, webcamsOnlyForModerator: Boolean, guestPolicy: String, allowModsToUnmuteUsers: Boolean, authenticatedGuest: Boolean)
case class MetadataProp(metadata: collection.immutable.Map[String, String])
diff --git a/bbb-common-message/src/test/scala/org/bigbluebutton/common2/TestFixtures.scala b/bbb-common-message/src/test/scala/org/bigbluebutton/common2/TestFixtures.scala
index 158a4a324c9231889b45fa466d72529879398b7e..127f0e6cb6db2e6210af47e633408078e0bfaa1a 100755
--- a/bbb-common-message/src/test/scala/org/bigbluebutton/common2/TestFixtures.scala
+++ b/bbb-common-message/src/test/scala/org/bigbluebutton/common2/TestFixtures.scala
@@ -35,6 +35,7 @@ trait TestFixtures {
val allowModsToUnmuteUsers = false
val keepEvents = false
val guestPolicy = "ALWAYS_ASK"
+ val authenticatedGuest = false
val metadata: collection.immutable.Map[String, String] = Map("foo" -> "bar", "bar" -> "baz", "baz" -> "foo")
val meetingProp = MeetingProp(name = meetingName, extId = externalMeetingId, intId = meetingId,
@@ -51,7 +52,7 @@ trait TestFixtures {
modOnlyMessage = modOnlyMessage)
val voiceProp = VoiceProp(telVoice = voiceConfId, voiceConf = voiceConfId, dialNumber = dialNumber, muteOnStart = muteOnStart)
val usersProp = UsersProp(maxUsers = maxUsers, webcamsOnlyForModerator = webcamsOnlyForModerator,
- guestPolicy = guestPolicy, allowModsToUnmuteUsers = allowModsToUnmuteUsers)
+ guestPolicy = guestPolicy, allowModsToUnmuteUsers = allowModsToUnmuteUsers, authenticatedGuest = authenticatedGuest)
val metadataProp = new MetadataProp(metadata)
val screenshareProps = ScreenshareProps(screenshareConf = "FixMe!", red5ScreenshareIp = "fixMe!",
red5ScreenshareApp = "fixMe!")
diff --git a/bbb-common-web/src/main/java/org/bigbluebutton/api/MeetingService.java b/bbb-common-web/src/main/java/org/bigbluebutton/api/MeetingService.java
index 54d3034e77284394c07210271932c2b5fb6eec9b..f09b5366ce0ec45e2f31d437f03dc15ab6ae7288 100755
--- a/bbb-common-web/src/main/java/org/bigbluebutton/api/MeetingService.java
+++ b/bbb-common-web/src/main/java/org/bigbluebutton/api/MeetingService.java
@@ -396,7 +396,7 @@ public class MeetingService implements MessageListener {
m.getTelVoice(), m.getDuration(), m.getAutoStartRecording(), m.getAllowStartStopRecording(),
m.getWebcamsOnlyForModerator(), m.getModeratorPassword(), m.getViewerPassword(), m.getCreateTime(),
formatPrettyDate(m.getCreateTime()), m.isBreakout(), m.getSequence(), m.isFreeJoin(), m.getMetadata(),
- m.getGuestPolicy(), m.getWelcomeMessageTemplate(), m.getWelcomeMessage(), m.getModeratorOnlyMessage(),
+ m.getGuestPolicy(), m.getAuthenticatedGuest(), m.getWelcomeMessageTemplate(), m.getWelcomeMessage(), m.getModeratorOnlyMessage(),
m.getDialNumber(), m.getMaxUsers(),
m.getMeetingExpireIfNoUserJoinedInMinutes(), m.getmeetingExpireWhenLastUserLeftInMinutes(),
m.getUserInactivityInspectTimerInMinutes(), m.getUserInactivityThresholdInMinutes(),
diff --git a/bbb-common-web/src/main/java/org/bigbluebutton/api/ParamsProcessorUtil.java b/bbb-common-web/src/main/java/org/bigbluebutton/api/ParamsProcessorUtil.java
index 732ea5fefce0d9fb0777729bbded0f0fe613879a..9ad48b113c492c28965f6276074f9f348da3c2a9 100755
--- a/bbb-common-web/src/main/java/org/bigbluebutton/api/ParamsProcessorUtil.java
+++ b/bbb-common-web/src/main/java/org/bigbluebutton/api/ParamsProcessorUtil.java
@@ -79,6 +79,7 @@ public class ParamsProcessorUtil {
private String defaultAvatarURL;
private String defaultConfigURL;
private String defaultGuestPolicy;
+ private Boolean authenticatedGuest;
private int defaultMeetingDuration;
private boolean disableRecordingDefault;
private boolean autoStartRecording;
@@ -485,6 +486,7 @@ public class ParamsProcessorUtil {
.withWelcomeMessageTemplate(welcomeMessageTemplate)
.withWelcomeMessage(welcomeMessage).isBreakout(isBreakout)
.withGuestPolicy(guestPolicy)
+ .withAuthenticatedGuest(authenticatedGuest)
.withBreakoutRoomsParams(breakoutParams)
.withLockSettingsParams(lockSettingsParams)
.withAllowDuplicateExtUserid(defaultAllowDuplicateExtUserid)
@@ -956,6 +958,10 @@ public class ParamsProcessorUtil {
this.defaultGuestPolicy = guestPolicy;
}
+ public void setAuthenticatedGuest(Boolean value) {
+ this.authenticatedGuest = value;
+ }
+
public void setClientLogoutTimerInMinutes(Integer value) {
clientLogoutTimerInMinutes = value;
}
diff --git a/bbb-common-web/src/main/java/org/bigbluebutton/api/domain/Meeting.java b/bbb-common-web/src/main/java/org/bigbluebutton/api/domain/Meeting.java
index f11e00d2b0e59088e79c72df30be5e09335ff757..1b4e1a6afb930a4319c807a64cafe16504ccd7e1 100755
--- a/bbb-common-web/src/main/java/org/bigbluebutton/api/domain/Meeting.java
+++ b/bbb-common-web/src/main/java/org/bigbluebutton/api/domain/Meeting.java
@@ -68,6 +68,7 @@ public class Meeting {
private String defaultAvatarURL;
private String defaultConfigToken;
private String guestPolicy = GuestPolicy.ASK_MODERATOR;
+ private Boolean authenticatedGuest = false;
private boolean userHasJoined = false;
private Map<String, String> metadata;
private Map<String, Object> userCustomData;
@@ -125,6 +126,7 @@ public class Meeting {
createdTime = builder.createdTime;
isBreakout = builder.isBreakout;
guestPolicy = builder.guestPolicy;
+ authenticatedGuest = builder.authenticatedGuest;
breakoutRoomsParams = builder.breakoutRoomsParams;
lockSettingsParams = builder.lockSettingsParams;
allowDuplicateExtUserid = builder.allowDuplicateExtUserid;
@@ -362,8 +364,36 @@ public class Meeting {
return guestPolicy;
}
+ public void setAuthenticatedGuest(Boolean authGuest) {
+ authenticatedGuest = authGuest;
+ }
+
+ public Boolean getAuthenticatedGuest() {
+ return authenticatedGuest;
+ }
+
+ private String getUnauthenticatedGuestStatus(Boolean guest) {
+ if (guest) {
+ switch(guestPolicy) {
+ case GuestPolicy.ALWAYS_ACCEPT:
+ case GuestPolicy.ALWAYS_ACCEPT_AUTH:
+ return GuestPolicy.ALLOW;
+ case GuestPolicy.ASK_MODERATOR:
+ return GuestPolicy.WAIT;
+ case GuestPolicy.ALWAYS_DENY:
+ return GuestPolicy.DENY;
+ default:
+ return GuestPolicy.DENY;
+ }
+ } else {
+ return GuestPolicy.ALLOW;
+ }
+ }
public String calcGuestStatus(String role, Boolean guest, Boolean authned) {
+ // Good ol' Mconf guest status
+ if (!authenticatedGuest) return getUnauthenticatedGuestStatus(guest);
+
// Allow moderators all the time.
if (ROLE_MODERATOR.equals(role)) {
return GuestPolicy.ALLOW;
@@ -670,6 +700,7 @@ public class Meeting {
private long createdTime;
private boolean isBreakout;
private String guestPolicy;
+ private Boolean authenticatedGuest;
private BreakoutRoomsParams breakoutRoomsParams;
private LockSettingsParams lockSettingsParams;
private Boolean allowDuplicateExtUserid;
@@ -791,6 +822,11 @@ public class Meeting {
guestPolicy = policy;
return this;
}
+
+ public Builder withAuthenticatedGuest(Boolean authGuest) {
+ authenticatedGuest = authGuest;
+ return this;
+ }
public Builder withBreakoutRoomsParams(BreakoutRoomsParams params) {
breakoutRoomsParams = params;
diff --git a/bbb-common-web/src/main/java/org/bigbluebutton/api2/IBbbWebApiGWApp.java b/bbb-common-web/src/main/java/org/bigbluebutton/api2/IBbbWebApiGWApp.java
index 4f9576646d819b6d0bd700093846c7b101763b08..76b12060b85809b702b2e3fa687956be7a4f953a 100755
--- a/bbb-common-web/src/main/java/org/bigbluebutton/api2/IBbbWebApiGWApp.java
+++ b/bbb-common-web/src/main/java/org/bigbluebutton/api2/IBbbWebApiGWApp.java
@@ -19,7 +19,7 @@ public interface IBbbWebApiGWApp {
Boolean allowStartStopRecording, Boolean webcamsOnlyForModerator,
String moderatorPass, String viewerPass, Long createTime,
String createDate, Boolean isBreakout, Integer sequence, Boolean freejoin, Map<String, String> metadata,
- String guestPolicy, String welcomeMsgTemplate, String welcomeMsg, String modOnlyMessage,
+ String guestPolicy, Boolean authenticatedGuest, String welcomeMsgTemplate, String welcomeMsg, String modOnlyMessage,
String dialNumber, Integer maxUsers,
Integer meetingExpireIfNoUserJoinedInMinutes,
Integer meetingExpireWhenLastUserLeftInMinutes,
diff --git a/bbb-common-web/src/main/java/org/bigbluebutton/api2/domain/UsersProp2.java b/bbb-common-web/src/main/java/org/bigbluebutton/api2/domain/UsersProp2.java
index bcb7016d80448a5e7ab15d0ff9c691ca5fb4e857..5c5a3fe069dd5b440b4ff1f7ef2f347a1920fe5e 100755
--- a/bbb-common-web/src/main/java/org/bigbluebutton/api2/domain/UsersProp2.java
+++ b/bbb-common-web/src/main/java/org/bigbluebutton/api2/domain/UsersProp2.java
@@ -5,6 +5,7 @@ import java.util.Map;
public class UsersProp2 {
public final String guestPolicy;
+ public final boolean authenticatedGuest;
public final boolean userHasJoined;
public final boolean webcamsOnlyForModerator;
public final int maxUsers;
@@ -15,6 +16,7 @@ public class UsersProp2 {
public UsersProp2(int maxUsers,
boolean webcamsOnlyForModerator,
String guestPolicy,
+ boolean authenticatedGuest,
boolean userHasJoined,
Map<String, String> userCustomData,
Map<String, User2> users,
@@ -22,6 +24,7 @@ public class UsersProp2 {
this.maxUsers = maxUsers;
this.webcamsOnlyForModerator = webcamsOnlyForModerator;
this.guestPolicy = guestPolicy;
+ this.authenticatedGuest = authenticatedGuest;
this.userHasJoined = userHasJoined;
this.userCustomData = userCustomData;
this.users = users;
diff --git a/bbb-common-web/src/main/scala/org/bigbluebutton/api2/BbbWebApiGWApp.scala b/bbb-common-web/src/main/scala/org/bigbluebutton/api2/BbbWebApiGWApp.scala
index ca536f6ac24cd841ded408f0fa78d0c442e01d6b..01ce305861261e3cbea15633ee2b0d990963f8a9 100755
--- a/bbb-common-web/src/main/scala/org/bigbluebutton/api2/BbbWebApiGWApp.scala
+++ b/bbb-common-web/src/main/scala/org/bigbluebutton/api2/BbbWebApiGWApp.scala
@@ -127,7 +127,7 @@ class BbbWebApiGWApp(
viewerPass: String, createTime: java.lang.Long, createDate: String, isBreakout: java.lang.Boolean,
sequence: java.lang.Integer,
freeJoin: java.lang.Boolean,
- metadata: java.util.Map[String, String], guestPolicy: String,
+ metadata: java.util.Map[String, String], guestPolicy: String, authenticatedGuest: java.lang.Boolean,
welcomeMsgTemplate: String, welcomeMsg: String, modOnlyMessage: String,
dialNumber: String, maxUsers: java.lang.Integer,
meetingExpireIfNoUserJoinedInMinutes: java.lang.Integer,
@@ -172,7 +172,7 @@ class BbbWebApiGWApp(
modOnlyMessage = modOnlyMessage)
val voiceProp = VoiceProp(telVoice = voiceBridge, voiceConf = voiceBridge, dialNumber = dialNumber, muteOnStart = muteOnStart.booleanValue())
val usersProp = UsersProp(maxUsers = maxUsers.intValue(), webcamsOnlyForModerator = webcamsOnlyForModerator.booleanValue(),
- guestPolicy = guestPolicy, allowModsToUnmuteUsers = allowModsToUnmuteUsers.booleanValue())
+ guestPolicy = guestPolicy, allowModsToUnmuteUsers = allowModsToUnmuteUsers.booleanValue(), authenticatedGuest = authenticatedGuest.booleanValue())
val metadataProp = MetadataProp(mapAsScalaMap(metadata).toMap)
val screenshareProps = ScreenshareProps(
screenshareConf = voiceBridge + screenshareConfSuffix,
diff --git a/bigbluebutton-html5/imports/api/meetings/server/modifiers/addMeeting.js b/bigbluebutton-html5/imports/api/meetings/server/modifiers/addMeeting.js
index 389d816a62a1037b4c7266b7a900e32cdd3b7874..9acea4e998f7e6fd828b69a48361c7dd374a93c8 100755
--- a/bigbluebutton-html5/imports/api/meetings/server/modifiers/addMeeting.js
+++ b/bigbluebutton-html5/imports/api/meetings/server/modifiers/addMeeting.js
@@ -35,6 +35,7 @@ export default function addMeeting(meeting) {
usersProp: {
webcamsOnlyForModerator: Boolean,
guestPolicy: String,
+ authenticatedGuest: Boolean,
maxUsers: Number,
allowModsToUnmuteUsers: Boolean,
},
diff --git a/bigbluebutton-html5/imports/ui/components/waiting-users/component.jsx b/bigbluebutton-html5/imports/ui/components/waiting-users/component.jsx
index 658ec84000e5772b727da0b8b8e8c77b096b6ff1..62e87853f21bb68652e180eba6f11a11610ae456 100755
--- a/bigbluebutton-html5/imports/ui/components/waiting-users/component.jsx
+++ b/bigbluebutton-html5/imports/ui/components/waiting-users/component.jsx
@@ -1,3 +1,4 @@
+import _ from 'lodash';
import React, { useEffect, useState } from 'react';
import { Session } from 'meteor/session';
import { defineMessages, injectIntl } from 'react-intl';
@@ -148,6 +149,7 @@ const WaitingUsers = (props) => {
guestUsers,
guestUsersCall,
changeGuestPolicy,
+ authenticatedGuest,
} = props;
const onCheckBoxChange = (e) => {
@@ -173,7 +175,7 @@ const WaitingUsers = (props) => {
/>
);
- const buttonsData = [
+ const authGuestButtonsData = [
{
messageId: intlMessages.allowAllAuthenticated,
action: () => guestUsersCall(authenticatedUsers, ALLOW_STATUS),
@@ -189,6 +191,9 @@ const WaitingUsers = (props) => {
key: 'allow-all-guest',
policy: 'ALWAYS_ACCEPT',
},
+ ];
+
+ const guestButtonsData = [
{
messageId: intlMessages.allowEveryone,
action: () => guestUsersCall([...guestUsers, ...authenticatedUsers], ALLOW_STATUS),
@@ -203,6 +208,8 @@ const WaitingUsers = (props) => {
},
];
+ const buttonsData = authenticatedGuest ? _.concat(authGuestButtonsData , guestButtonsData) : guestButtonsData;
+
return (
<div
data-test="note"
diff --git a/bigbluebutton-html5/imports/ui/components/waiting-users/container.jsx b/bigbluebutton-html5/imports/ui/components/waiting-users/container.jsx
index b3fe25c21d16d87eb2e88f28613cc3f1b7143112..7e6e879ae4038f9091894070e8b581c7fb7b2bee 100644
--- a/bigbluebutton-html5/imports/ui/components/waiting-users/container.jsx
+++ b/bigbluebutton-html5/imports/ui/components/waiting-users/container.jsx
@@ -2,6 +2,7 @@ import React, { PureComponent } from 'react';
import { withTracker } from 'meteor/react-meteor-data';
import Auth from '/imports/ui/services/auth';
import GuestUsers from '/imports/api/guest-users/';
+import Meetings from '/imports/api/meetings';
import Service from './service';
import WaitingComponent from './component';
@@ -30,10 +31,13 @@ export default withTracker(() => {
denied: false,
}).fetch();
+ const authenticatedGuest = Meetings.findOne({ meetingId: Auth.meetingID }).usersProp.authenticatedGuest;
+
return {
guestUsers,
authenticatedUsers,
guestUsersCall: Service.guestUsersCall,
changeGuestPolicy: Service.changeGuestPolicy,
+ authenticatedGuest,
};
})(WaitingContainer);
diff --git a/bigbluebutton-web/grails-app/conf/bigbluebutton.properties b/bigbluebutton-web/grails-app/conf/bigbluebutton.properties
index 6979e3fb5dc343faeacd35be4abeb1d4683ab552..1fc1c8e9e69035e3d7438851909947ff1d173111 100755
--- a/bigbluebutton-web/grails-app/conf/bigbluebutton.properties
+++ b/bigbluebutton-web/grails-app/conf/bigbluebutton.properties
@@ -148,6 +148,9 @@ defaultDialAccessNumber=613-555-1234
#
defaultGuestPolicy=ALWAYS_ACCEPT
+# Enables or disables authenticated guest
+authenticatedGuest=false
+
#
#----------------------------------------------------
# Default welcome message to display when the participant joins the web
diff --git a/bigbluebutton-web/grails-app/conf/spring/resources.xml b/bigbluebutton-web/grails-app/conf/spring/resources.xml
index 34d77afe8b5433aa4cba62ed45c92554d3dd8977..c03ef9d2a86350ba2ed26999344462159114bb68 100755
--- a/bigbluebutton-web/grails-app/conf/spring/resources.xml
+++ b/bigbluebutton-web/grails-app/conf/spring/resources.xml
@@ -139,6 +139,7 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
<property name="defaultAvatarURL" value="${defaultAvatarURL}"/>
<property name="defaultConfigURL" value="${defaultConfigURL}"/>
<property name="defaultGuestPolicy" value="${defaultGuestPolicy}"/>
+ <property name="authenticatedGuest" value="${authenticatedGuest}"/>
<property name="meetingExpireIfNoUserJoinedInMinutes" value="${meetingExpireIfNoUserJoinedInMinutes}"/>
<property name="meetingExpireWhenLastUserLeftInMinutes" value="${meetingExpireWhenLastUserLeftInMinutes}"/>
<property name="userInactivityInspectTimerInMinutes" value="${userInactivityInspectTimerInMinutes}"/>