Subcategories do not inherit permissions from parent category

A very good point. And good to hear “out loud”. This is the right way for me to be thinking about confidentiality and privacy.

5 „Gefällt mir“

I have used private categories with “less private” subcategories, so that a group can work in their space and publish things to a larger public. I guess this line of work will become obsolete by this change. So I should let users know that when they want to publish something, they need to move it to another, more public category. But then, how do they find their common work on this? I suppose they need to maintain a “publications” topic where they link the ‘more public’ topics. Any thoughts?

1 „Gefällt mir“

I am reviewing FIX: validate parent category/subcategories permissions by majakomel · Pull Request #6877 · discourse/discourse · GitHub by @maja.

The particular condition we are protecting against is:

- category (authors: read)
-    subcategory (pilots: read)

What this means is that pilots are not allowed to see anything in the category yet have been granted permissions in the subcategory so something is fishy :fish:

The condition:

- category (authors: read)
-    subcategory (pilots: read, authors: write)

Is still :ok: after this new validation.

The tricky thing though :llama: is that we no longer will allow:

- category (trust_level_3: read)
-   subcategory (trust_level_4: write)

This particular edge case gives me pause, there is an easy enough work around … you would have to explicitly change

- category (trust_level_3: read, trust_level_4: read)

Given the OP is in a pretty bad state I say we see how the new restriction shakes up. FYI @HAWK @jomaxro

10 „Gefällt mir“

@sam Ich sehe, dass der PR gemerged wurde, aber ich bin mir nicht sicher, ob die Funktionalität für Vererbung in unserer Discourse-Installation v2.4.0.beta2 +183 berücksichtigt wird?

Wir waren ziemlich schockiert festzustellen, dass der gesamte Inhalt, von dem wir dachten, er sei in Unterkategorien privat, sofort für die nicht authentifizierten Öffentlichkeit über latest verfügbar war.

Da wir fast 10 Kategorien haben, jede mit zahlreichen Unterkategorien, wäre es wirklich umständlich, die Berechtigungen manuell für jede einzelne einzustellen.

1 „Gefällt mir“

@sam Es scheint jetzt zu funktionieren, aber es gibt anscheinend noch einen Eckenfall (der uns heute etwas Ärger bereitet hat :frowning: ).

Es ist möglich, beim Erstellen der Unterkategorie auf dem Reiter „Sicherheit

Um zu sehen, ob diese Option irgendwo aktiviert wurde? Eine Möglichkeit, die Berechtigungen für Kategorien zu ändern und alle Unterkategorien diese neue Berechtigung erben zu lassen?