0
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2026-08-06 13:08:40 +08:00
discourse/app/controllers/admin/groups_controller.rb
Régis Hanol ac8d3b0515
UX: Normalize automatic membership email domains (#41434)
Previously, entering an automatic membership email domain that included
an `@` (such as `@acme.io` or a pasted email or URL) warned the admin
and blocked the save until they hand-edited it down to a bare domain.

This change quietly normalizes those entries to the bare domain —
stripping any scheme, path, port, `@` prefix, casing, and surrounding
whitespace — so common paste mistakes just work, while the server still
rejects genuinely invalid domains on save.

Follow up to #41233
2026-07-03 17:22:18 +02:00

151 lines
4.2 KiB
Ruby
Vendored

# frozen_string_literal: true
class Admin::GroupsController < Admin::StaffController
MAX_AUTO_MEMBERSHIP_DOMAINS_LOOKUP = 10
def index
end
def create
Groups::Create.call(
guardian:,
params: group_params.except(*DiscoursePluginRegistry.group_params),
options: {
dynamic_attributes: group_params.slice(*DiscoursePluginRegistry.group_params),
},
) do |result|
on_success { |group:| render_serialized(group, BasicGroupSerializer) }
on_failed_policy(:can_create_group) { |policy| raise Discourse::InvalidAccess }
on_failed_policy(:can_request_access) do
render json:
failed_json.merge(
errors: [I18n.t("groups.errors.cant_allow_membership_requests")],
),
status: :unprocessable_entity
end
on_model_errors(:group) { |group:| render_json_error(group) }
on_failure { render(json: failed_json, status: :unprocessable_entity) }
end
end
def destroy
group = Group.find_by(id: params[:id])
raise Discourse::NotFound unless group
if group.automatic
can_not_modify_automatic
else
StaffActionLogger.new(current_user).log_group_deletion(group)
group.destroy!
render json: success_json
end
end
def remove_owner
group = Group.find_by(id: params.require(:id))
raise Discourse::NotFound unless group
return can_not_modify_automatic if group.automatic
guardian.ensure_can_edit_group!(group)
if params[:user_id].present?
users = [User.find_by(id: params[:user_id].to_i)]
elsif usernames = group_params[:usernames].presence
users = User.where(username: usernames.split(","))
else
raise Discourse::InvalidParameters.new(:user_id)
end
users.each do |user|
group.group_users.where(user_id: user.id).update_all(owner: false)
GroupActionLogger.new(current_user, group).log_remove_user_as_group_owner(user)
end
render json: success_json
end
def set_primary
group = Group.find_by(id: params.require(:id))
raise Discourse::NotFound unless group
users = User.where(username: group_params[:usernames].split(","))
users.each { |user| guardian.ensure_can_change_primary_group!(user, group) }
users.update_all(primary_group_id: params[:primary] == "true" ? group.id : nil)
render json: success_json
end
def automatic_membership_count
guardian.ensure_can_create_group!
domains = Group.get_valid_email_domains(params.require(:automatic_membership_email_domains))
group_id = params[:id]
user_count = 0
if domains.present?
if group_id.present?
group = Group.find_by(id: group_id)
raise Discourse::NotFound unless group
return can_not_modify_automatic if group.automatic
existing_domains = group.automatic_membership_email_domains&.split("|") || []
domains -= existing_domains
end
if domains.size > MAX_AUTO_MEMBERSHIP_DOMAINS_LOOKUP
user_count = nil
else
user_count = Group.automatic_membership_users(domains.join("|")).count
end
end
render json: { user_count: }
end
protected
def can_not_modify_automatic
render_json_error(I18n.t("groups.errors.can_not_modify_automatic"))
end
private
def group_params
permitted = %i[
name
mentionable_level
messageable_level
visibility_level
members_visibility_level
automatic_membership_email_domains
title
primary_group
grant_trust_level
incoming_email
flair_icon
flair_upload_id
flair_bg_color
flair_color
bio_raw
public_admission
public_exit
allow_membership_requests
full_name
default_notification_level
membership_request_template
owner_usernames
usernames
publish_read_state
notify_users
]
custom_fields = DiscoursePluginRegistry.editable_group_custom_fields
permitted << { custom_fields: custom_fields } if custom_fields.present?
permitted << { associated_group_ids: [] } if guardian.can_associate_groups?
permitted = permitted | DiscoursePluginRegistry.group_params
params.require(:group).permit(permitted)
end
end