Merge branch 'v2'

This commit is contained in:
jfederico 2020-04-22 23:40:28 -04:00
commit aeaa1e7d45
2 changed files with 18 additions and 2 deletions

View File

@ -62,7 +62,8 @@ module Joiner
redirect_to join_path(@room, current_user.name, opts, current_user.uid) redirect_to join_path(@room, current_user.name, opts, current_user.uid)
else else
join_name = params[:join_name] || params[@room.invite_path][:join_name] join_name = params[:join_name] || params[@room.invite_path][:join_name]
redirect_to join_path(@room, join_name, opts)
redirect_to join_path(@room, join_name, opts, fetch_guest_id)
end end
else else
search_params = params[@room.invite_path] || params search_params = params[@room.invite_path] || params
@ -92,4 +93,19 @@ module Joiner
recording_default_visibility: @settings.get_value("Default Recording Visibility") == "public" recording_default_visibility: @settings.get_value("Default Recording Visibility") == "public"
} }
end end
private
def fetch_guest_id
return cookies[:guest_id] if cookies[:guest_id].present?
guest_id = "gl-guest-#{SecureRandom.hex(12)}"
cookies[:guest_id] = {
value: guest_id,
expires: 1.day.from_now
}
guest_id
end
end end

View File

@ -248,7 +248,7 @@ describe RoomsController, type: :controller do
allow_any_instance_of(BigBlueButton::BigBlueButtonApi).to receive(:is_meeting_running?).and_return(true) allow_any_instance_of(BigBlueButton::BigBlueButtonApi).to receive(:is_meeting_running?).and_return(true)
post :join, params: { room_uid: @room, join_name: "Join Name" } post :join, params: { room_uid: @room, join_name: "Join Name" }
expect(response).to redirect_to(join_path(@owner.main_room, "Join Name", {})) expect(response).to redirect_to(join_path(@owner.main_room, "Join Name", {}, response.cookies["guest_id"]))
end end
it "should render wait if meeting isn't running" do it "should render wait if meeting isn't running" do