Repository navigation
Show public channels and posting requirements on homepage - #224
LIghtJUNction wants to merge 2 commits into
Conversation
|
You have reached your Codex usage limits for security reviews. Please try again later. |
|
Integrated into main c57511c and deployed as msgd 0.2.3-20261001.1. Live homepage now lists 10 public channels with posting and certificate requirements; private channels are omitted. Targeted PostgreSQL tests and live /install smoke check passed. |
|
[P2] Account for /store’s controlled namespace in the posting guidance At f4de758, /store has the default mode 1777, so the new mode-based branch advertises “Authenticated identity and authorization to create posts.” However, ordinary content.post_create calls there always reach store_controlled_resource: create_resource only permits listing resources created through store.listing_create or bounty.create. An authenticated caller with ordinary create permission therefore cannot do what the homepage suggests. The new test also sets /store to 0755 and expects “Channel owner,” but ownership does not remove this namespace restriction. Please give /store dedicated listing/bounty guidance, as /last-will already has for its special operation, or exclude it from discussion channels. Cover both the default and owner-only modes using the appropriate expected guidance. Source evidence: discovery.py lines 636–649 ( msg/src/msg/plugins/discovery.py Lines 636 to 649 in f4de758 Lines 146 to 152 in f4de758 msg/tests/test_store_foundation.py Lines 255 to 265 in f4de758 |
The homepage now lists active, publicly readable top-level discussion channels with read and posting requirements. Private, archived and internal channels are omitted through current resource-read authorization; no authenticated read ceiling or operation version changes.
Posting guidance distinguishes authenticated creation, owner/group permissions, scoped certified-write certificates, frozen topics and signed legacy directives. It does not expose member IDs or certificate/ACL contents.
Validation: 8 PostgreSQL-backed homepage and permission/rendering tests passed; Ruff and diff checks passed. Rebased on the installation/promotion release, retaining /install.