La strada verso la stabilità, poi la permanenza, per granular_anonymous_and_logged_in_groups_permissions

[martin (Martin Brennan)]

Questo è un argomento correlato a Granular group-based permissions for anonymous and logged in users ed è in fase di sviluppo (WIP).

---

Ci sono ancora alcuni punti in vari codebase per core, plugin e temi che fanno riferimento diretto al gruppo everyone o all’ID del gruppo (0), senza tenere conto di questa imminente modifica o senza utilizzare user.in_any_groups? o i vari metodi guardian utilizzati per gestirlo.

Per riferimento rapido, alcuni ID di gruppo automatici:

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

Questo argomento funge da punto centrale per tenere traccia delle correzioni per questo sistema man mano che ci avviciniamo alla versione stabile. I plugin e i temi personalizzati per i clienti di Discourse non sono tracciati qui; avrò un elenco separato internamente per questo.

Problemi ad alta priorità

Problema sollevato da Moin:

@martin

1. anonymous_users e logged_in_users non sono presenti nel selettore di gruppi. Credo sia sicuro consentire questi qui ora, in modo che non farà differenza se hai aggiunto everyone a disallowed_groups se la modifica imminente è disattivata.
2. Correggere Guardian::AnonymousUser#in_any_groups? per rispettare anonymous_users con la modifica imminente disattivata.
3. Aggiungere lo stesso aliasing del tempo di lettura di 0 (everyone) → 5 (logged_in_users) per le impostazioni dei temi, come facciamo per le impostazioni del sito.

• Core
  • About - apply_excluded_groups viene utilizzato solo per nascondere i moderatori e gli amministratori selezionati dai gruppi in about_page_hidden_groups. Non ha senso che i pseudogruppi come 0|4|5 vengano utilizzati qui; dovrebbero essere aggiunti a disallowed_groups. Dobbiamo anche aggiornare la descrizione dell’impostazione poiché è fuorviante.
  • EditCategoryTabsController - In _wouldLoseAccess discourse/frontend/discourse/admin/controllers/edit-category/tabs.js at 86552acfbaf816db3b9db5f761f5e9f2bd221c21 · discourse/discourse · GitHub stiamo guardando solo i gruppi visibili; questo dovrebbe essere un controllo lato server come evaluate per gli ACL

Plugin core

• Discourse Templates
  • • can_use_private_templates? fa ancora riferimento diretto a everyone e non utilizza la notazione abbreviata _map nelle impostazioni del sito.
• Discourse AI
  • • can_see_summary? nelle estensioni guardian non utilizza user.in_any_groups?
  • • can_attach? in AiBot::Playground non utilizza user.in_any_groups?
  • • addTopicAdminMenuButton in ai-translation-topic-admin guarda currentUser.groups lato client, il che non è affidabile; eseguire invece un controllo lato server per content_localization_allowed_groups
• Discourse Assign. Abbastanza tanti problemi qui.
  • Invece di user_ids_in_groups in AssignmentPermissions, dovremmo usare user_ids_in_groups
  • assign_allowed_on_groups dovrebbe avere 0|4|5 aggiunti a disallowed_groups; nessuno di questi ha senso qui, contano solo i gruppi concreti.

Altri plugin

• Activity Pub
  • Rimuovere il controllo lato client di showStatusToUser su user.groups e il controllo di everyone. Cambiare activity_pub_post_status_visibility_groups per impostazione predefinita su 4|5.
• Suggested Edits
  • user_in_suggested_edits_group? deve usare user.in_any_groups? in GuardianExtensions. Aggiungere 1 a mandatory_groups per suggested_edits_review_groups e rimuovere il caso speciale per gli amministratori in can_review_suggested_edits_in_topic_list
  • Aggiungere 0|4|5 a disallowed_groups per suggested_edits_review_groups e suggested_edits_suggest_groups; questa impostazione non è realmente destinata all’uso con questi pseudogruppi
• Resenha
  • Rimuovere la workaround untargetable_group_ids in public_room_message_bus_targets poiché questo è stato affrontato in FIX: Handle new pseudogroups in message bus group IDs (#42610) · discourse/discourse@699ad46 · GitHub
  • Cambiare l’impostazione predefinita di resenha_allowed_groups da 0 a 4|5

Temi

Componenti

Problemi a bassa priorità

• Core
  • is_in_edit_topic_groups in TopicGuardian dovrebbe usare l’estensione _map per l’impostazione del sito
  • Roleable#whisperer? dovrebbe usare user.in_any_groups? e eliminare il controllo per gli amministratori, aggiungere 1 all’impostazione whispers_allowed_groups come mandatory_values
  • Aggiungere 0|4 a disallowed_groups per l’impostazione del sito allow_solved_in_groups; questo riguarda solo i PM
  • Aggiungere 0|4|5 a disallowed_groups per about_page_extra_groups; questo è destinato a gestire solo gruppi concreti nella pagina about
  • Nota in can_view? per PresenceChannel che dobbiamo rimuovere il controllo di Group::AUTO_GROUPS[:everyone] quando questa imminente modifica passa a Permanent

Plugin core

• Chat
  • Nota per everyone_allowed in vari punti nella logica di auto-join/auto-leave di chat e users_with_unreads per rimuovere il controllo di :everyone quando questa imminente modifica passa a Permanent
  • Aggiornare chat_allowed_group_ids in Chat::Publisher per rimuovere lo pseudogruppo speciale, poiché come in FIX: Handle new pseudogroups in message bus group IDs - Pull Request #42610 - discourse/discourse - GitHub MessageBus può ora gestire correttamente gli pseudogruppi
• Assign
  • Eliminare add_model_callback(Group) in plugin.rb, è codice irrilevante/morto che guarda i nomi dei gruppi invece degli ID

Altri plugin

• Code Review
  • Il controllo di can_review_code? su code_review_allowed_groups non utilizza l’estensione _map e non utilizza user.in_any_groups?. Aggiungere 1 a mandatory_groups per l’impostazione ed eliminare il controllo speciale per gli amministratori in guardian. Aggiungere 0|4|5 a disallowed_groups.
• Needs Love
  • Il controllo di can_needs_love? su needs_love_allowed_groups non utilizza l’estensione _map e non utilizza user.in_any_groups?. Aggiungere 1 a mandatory_groups per l’impostazione ed eliminare il controllo speciale per gli amministratori in guardian. Aggiungere 0|4|5 a disallowed_groups.
• Staff Alias
  • Il controllo di can_post_as_staff_alias su staff_alias_allowed_groups non utilizza l’estensione _map e non utilizza user.in_any_groups?. Aggiungere 0|4|5 a disallowed_groups

Temi

Componenti

Prossimi passi per la migrazione

• Migrare allowed_group_ids del modello AiAgent