O caminho para estabilidade e, em seguida, permanência de granular_anonymous_and_logged_in_groups_permissions

Dev
1

Este é um tópico complementar a Granular group-based permissions for anonymous and logged in users e está em desenvolvimento (WIP).

Ainda existem alguns lugares em diversas bases de código para o núcleo, plugins e temas que referenciam diretamente everyone ou o ID do grupo (0), sem levar em conta esta mudança iminente, ou sem usar user.in_any_groups? ou os vários métodos de guardian usados para lidar com isso.

Para referência rápida, alguns IDs de grupos automáticos:

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

Este tópico serve como um local central para acompanhar as correções deste sistema à medida que nos aproximamos da versão estável. Plugins e temas personalizados para clientes do Discourse não são rastreados aqui; terei uma lista separada internamente para isso.

Problemas de alta prioridade

Problema levantado por Moin:

Plugins do núcleo

  • Discourse Templates
      • can_use_private_templates? ainda referencia everyone diretamente e não usa o atalho _map nas configurações do site.
  • Discourse AI
      • can_see_summary? nas extensões de guardian não usando user.in_any_groups?
      • can_attach? em AiBot::Playground não usa user.in_any_groups?
      • addTopicAdminMenuButton em ai-translation-topic-admin olhando para currentUser.groups no cliente, o que não é confiável; faça uma verificação no servidor para content_localization_allowed_groups em vez disso
  • Discourse Assign. Bastante problemas aqui.
    • Em vez de user_ids_in_groups em AssignmentPermissions, devemos usar user_ids_in_groups
    • assign_allowed_on_groups deve ter 0|4|5 adicionado a disallowed_groups; nenhum desses faz sentido aqui, apenas grupos concretos importam.

Outros plugins

  • Activity Pub
    • Remover a verificação client-side de showStatusToUser em user.groups e a verificação de everyone. Mudar activity_pub_post_status_visibility_groups para padrão 4|5.
  • Suggested Edits
    • user_in_suggested_edits_group? precisa usar user.in_any_groups? em GuardianExtensions. Adicionar 1 a mandatory_groups em suggested_edits_review_groups e remover o caso especial de admin em can_review_suggested_edits_in_topic_list
    • Adicionar 0|4|5 a disallowed_groups para suggested_edits_review_groups e suggested_edits_suggest_groups; essa configuração não foi realmente destinada a ser usada com esses pseudogrupos
  • Live

Temas

Componentes

Problemas de baixa prioridade

  • Núcleo
    • is_in_edit_topic_groups em TopicGuardian deve usar a extensão _map para a configuração do site
    • Roleable#whisperer? deve usar user.in_any_groups? e se livrar da verificação de admin; adicionar 1 à configuração whispers_allowed_groups como mandatory_values
    • Adicionar 0|4 a disallowed_groups para a configuração do site allow_solved_in_groups; isso lida apenas com PMs
    • Adicionar 0|4|5 a disallowed_groups para about_page_extra_groups; isso deve lidar apenas com grupos concretos na página sobre
    • Anotar em can_view? para PresenceChannel que precisamos remover a verificação de Group::AUTO_GROUPS[:everyone] quando esta mudança iminente se mover para Permanent

Plugins do núcleo

  • Chat
  • Assign
    • Se livrar de add_model_callback(Group) em plugin.rb; é irrelevante/código morto olhando para nomes de grupos em vez de IDs

Outros plugins

  • Code Review
    • A verificação de can_review_code? em code_review_allowed_groups não usa a extensão _map e não usa user.in_any_groups?. Adicionar 1 a mandatory_groups para a configuração e se livrar da verificação especial de admin no guardian. Adicionar 0|4|5 a disallowed_groups.
  • Needs Love
    • A verificação de can_needs_love? em needs_love_allowed_groups não usa a extensão _map e não usa user.in_any_groups?. Adicionar 1 a mandatory_groups para a configuração e se livrar da verificação especial de admin no guardian. Adicionar 0|4|5 a disallowed_groups.
  • Staff Alias
    • A verificação de can_post_as_staff_alias em staff_alias_allowed_groups não usa a extensão _map e não usa user.in_any_groups?. Adicionar 0|4|5 a disallowed_groups

Temas

Componentes

Próximas etapas para migração

  • Migrar allowed_group_ids do modelo AiAgent
3

Concluído aqui FIX: everyone pseudogroup roadmap part 1 - Pull Request #43798 - discourse/discourse - GitHub