Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added passwordoption, advanced user interaction and changed subdir #1

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

svoeth
Copy link

@svoeth svoeth commented Apr 26, 2020

Expanded bbb-easy-join to accept user set passwords and to give some more insides if something went wrong. It is also possible to roll out bbb-easy-join next to greenlight by changing the subdirectory and default port.

You can test this version on senfcall.de (german version in german branch. Actual development in private gitlab - just copy and pasted current master here)

Passwordoption (a606fe2)

If no password is set the usabilty stays the same (1st user gets mod rights).

New:
If password ist set you have to provide the correct password to join. Only moderators will get the link to the meeting and also the password shown in welcome message. They will also get a second password which allows to join as moderator.
If the wrong password is provided an information will be shown.

Userinformation and operabilty (a606fe2)

New:
If you visit /b/RoomName and RoomName does not exist you will be informed about the missing room and you are able to create it with one click.

New:
If you try to create an existing room, you will be informed and asked if you want to create another room or join the existing room.

New:
In all three cases there is a slow delay of 1-2seconds to prevent simple bruteforce attacks (scriptkiddies). Won't hold off good attackers, but better than nothing :)

Added possibilty to run next to greenlight

New:
Changed default Port and subdirectory to make it possible to run bbb-easy-join next to greenlight

Room prefix

New:
(c6a2f72): Room now get an internal prefix ("easy-"). This way it is not possible to (accidentally) hijack a room which is for example "reserved" by greenlight.

(1cac515): bugfix

README and docs

(a606fe2): Adapted README and overview.png to inform about the new functions and also added some info. Github path in README has to be changed before merge (svoeth -> stadtulm)

(8ba5c3e): Expanded README for new features
(9dba216): Added instructions to change the sudirectory

@svoeth svoeth changed the title added passwordoption and advanced user interaction added passwordoption, advanced user interaction and changeable subdir Apr 26, 2020
@svoeth svoeth changed the title added passwordoption, advanced user interaction and changeable subdir added passwordoption, advanced user interaction and changed subdir Apr 26, 2020
@TheOneWithTheBraid
Copy link

I would love to see this in the upstream. On our BBB we successfully integrated your modifications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants