Skip to content

Commit

Permalink
LTI-380: remove hard coded BBB param prefix
Browse files Browse the repository at this point in the history
  • Loading branch information
Mariam05 committed Jun 19, 2024
1 parent 8aba2b5 commit 96d695e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion app/controllers/concerns/bbb_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ def add_ext_params(action, options)
@chosen_room.settings['ext_params']&.[](action)&.each do |key, value|
# the value in ext_params from the tenant settings is the name that should be passed to BBB
bbb_name = ext_params&.[](action)&.[](key)
options["userdata-#{bbb_name}"] = value if bbb_name
options[bbb_name] = value if bbb_name
end
end
end
2 changes: 2 additions & 0 deletions app/controllers/rooms_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -529,6 +529,8 @@ def set_ext_params
broker_ext_params = tenant_setting(tenant, 'ext_params')
lms_custom_params = launch_request_params['message']['custom_params']

logger.debug("[Rooms\'s Controller] extra params from broker: #{broker_ext_params} \n custom params from lms: #{lms_custom_params}")

pass_on_join_params = lms_custom_params.select { |k, _| broker_ext_params&.[]('join')&.key?(k) }
pass_on_create_params = lms_custom_params.select { |k, _| broker_ext_params&.[]('create')&.key?(k) }

Expand Down

0 comments on commit 96d695e

Please sign in to comment.