El camino hacia la estabilidad y luego la permanencia de granular_anonymous_and_logged_in_groups_permissions

[martin (Martin Brennan)]

Este es un tema complementario a Granular group-based permissions for anonymous and logged in users , y se encuentra en desarrollo (WIP).

---

Aún hay algunos lugares en varios bases de código para el núcleo, plugins y temas que se refieren directamente al grupo everyone o al ID de grupo (0), sin tener en cuenta este próximo cambio, o sin utilizar user.in_any_groups? o los diversos métodos de guardian utilizados para manejar esto.

Para referencia rápida, algunos IDs de grupos automáticos:

• 0 - everyone
• 1 - admins
• 2 - moderators
• 3 - staff
• 4 - anonymous_users
• 5 - logged_in_users

Este tema sirve como un lugar central para llevar un registro de las correcciones para este sistema a medida que nos acercamos a la versión estable. Los plugins y temas personalizados para los clientes de Discourse no se rastrean aquí; tendré una lista separada internamente para esto.

Problemas de alta prioridad

Problema planteado por Moin:

@martin

1. anonymous_users y logged_in_users no están en absoluto en el selector de grupos. Creo que es seguro permitir estos aquí ahora, de modo que no importará si has agregado everyone a disallowed_groups si el próximo cambio está desactivado.
2. Corregir Guardian::AnonymousUser#in_any_groups? para respetar anonymous_users con el próximo cambio desactivado.
3. Agregar el mismo alias de tiempo de lectura de 0 (everyone) → 5 (logged_in_users) para los ajustes de tema, tal como lo hacemos para los ajustes del sitio.

• Núcleo
  • Acerca de - apply_excluded_groups solo se utiliza para ocultar moderadores y administradores seleccionados de los grupos en about_page_hidden_groups. No tiene sentido que los seudogrupos como 0|4|5 se usen aquí, deberían agregarse a disallowed_groups. También necesitamos actualizar la descripción del ajuste, ya que es engañosa.
  • EditCategoryTabsController - En _wouldLoseAccess discourse/frontend/discourse/admin/controllers/edit-category/tabs.js at 86552acfbaf816db3b9db5f761f5e9f2bd221c21 · discourse/discourse · GitHub solo estamos mirando los grupos visibles; esto debería ser una verificación del lado del servidor, similar a evaluate para las ACL.

Plugins del núcleo

• Discourse Templates
  • • can_use_private_templates? aún se refiere directamente a everyone y no utiliza el atajo _map en los ajustes del sitio.
• Discourse AI
  • • can_see_summary? en las extensiones de guardian no utiliza user.in_any_groups?
  • • can_attach? en AiBot::Playground no utiliza user.in_any_groups?
  • • addTopicAdminMenuButton en ai-translation-topic-admin mira currentUser.groups en el cliente, lo cual no es confiable; haz una verificación del lado del servidor para content_localization_allowed_groups en su lugar.
• Discourse Assign. Hay bastantes problemas aquí.
  • En lugar de user_ids_in_groups en AssignmentPermissions, deberíamos usar user_ids_in_groups
  • assign_allowed_on_groups debería tener 0|4|5 agregado a disallowed_groups; ninguno de estos tiene sentido aquí, solo importan los grupos concretos.

Otros plugins

• Activity Pub
  • Eliminar la verificación del lado del cliente de user.groups y la verificación de everyone en showStatusToUser. Cambiar activity_pub_post_status_visibility_groups para que por defecto sea 4|5.
• Suggested Edits
  • user_in_suggested_edits_group? necesita usar user.in_any_groups? en GuardianExtensions. Agregar 1 a mandatory_groups de suggested_edits_review_groups y eliminar el caso especial de administrador en can_review_suggested_edits_in_topic_list.
  • Agregar 0|4|5 a disallowed_groups de suggested_edits_review_groups y suggested_edits_suggest_groups; este ajuste no está realmente diseñado para usarse con estos seudogrupos.
• Resenha
  • Eliminar el truco de untargetable_group_ids en public_room_message_bus_targets, ya que esto se abordó en FIX: Handle new pseudogroups in message bus group IDs (#42610) · discourse/discourse@699ad46 · GitHub
  • Cambiar el valor predeterminado de resenha_allowed_groups de 0 a 4|5.

Temas

Componentes

Problemas de baja prioridad

• Núcleo
  • is_in_edit_topic_groups en TopicGuardian debería usar la extensión _map para el ajuste del sitio.
  • Roleable#whisperer? debería usar user.in_any_groups? y deshacerse de la verificación de administrador; agregar 1 al ajuste whispers_allowed_groups como mandatory_values.
  • Agregar 0|4 a disallowed_groups para el ajuste del sitio allow_solved_in_groups; esto solo trata con mensajes privados.
  • Agregar 0|4|5 a disallowed_groups para about_page_extra_groups; esto está destinado a tratar solo con grupos concretos en la página de información.
  • Anotar en can_view? para PresenceChannel que necesitamos eliminar la verificación de Group::AUTO_GROUPS[:everyone] cuando este próximo cambio se mueva a Permanent.

Plugins del núcleo

• Chat
  • Anotar para everyone_allowed en varios lugares de la lógica de unirse/salirse automáticamente del chat y users_with_unreads para eliminar la verificación de :everyone cuando este próximo cambio se mueva a Permanent.
  • Actualizar chat_allowed_group_ids en Chat::Publisher para eliminar el seudogrupo especial, ya que en FIX: Handle new pseudogroups in message bus group IDs - Pull Request #42610 - discourse/discourse - GitHub MessageBus ahora puede manejar los seudogrupos correctamente.
• Assign
  • Deshacerse de add_model_callback(Group) en plugin.rb; es código irrelevante/muerto que mira nombres de grupos en lugar de IDs.

Otros plugins

• Code Review
  • La verificación de can_review_code? en code_review_allowed_groups no utiliza la extensión _map y no utiliza user.in_any_groups?. Agregar 1 a mandatory_groups para el ajuste y deshacerse de la verificación especial de administrador en guardian. Agregar 0|4|5 a disallowed_groups.
• Needs Love
  • La verificación de can_needs_love? en needs_love_allowed_groups no utiliza la extensión _map y no utiliza user.in_any_groups?. Agregar 1 a mandatory_groups para el ajuste y deshacerse de la verificación especial de administrador en guardian. Agregar 0|4|5 a disallowed_groups.
• Staff Alias
  • La verificación de can_post_as_staff_alias en staff_alias_allowed_groups no utiliza la extensión _map y no utiliza user.in_any_groups?. Agregar 0|4|5 a disallowed_groups.

Temas

Componentes

Próximos pasos para la migración

Por definir (TBA)