Skip to content

fix: show /run to every member, the relay's allowlist still refuses it - #82

Merged
Bilb merged 1 commit into
mainfrom
fix/run-visible
Nov 3, 2026
Merged

Bilb merged 1 commit into
mainfrom
fix/run-visible

Conversation

@Bilb

@Bilb Bilb commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator

Discord lets only someone who can use a command change its Integrations overrides. Registered with default_member_permissions: "0", /run was usable by administrators alone, so a non-admin on the allowlist could neither see it nor grant it to themselves.

  • /run is registered without default_member_permissions, as /versions is: shown to every member, in the channels the app's Integrations settings allow.
  • Who can run a job is unchanged: the relay refuses anyone outside ALLOWED_USER_IDS / ALLOWED_ROLE_IDS, administrators included, and ALLOWED_CHANNEL_IDS when set.
  • Docs and deploy/README.md step 6 updated.

After merging: git pull in /opt/session-ops and run session-ops discord-register again. No install.sh needed.

Tests

uv run --locked python -m unittest discover -s tests -t . (906 OK, 1 skipped), ruff check . clean.

@Bilb
Bilb merged commit 0e41c44 into main Nov 3, 2026
3 checks passed
@Bilb
Bilb deleted the fix/run-visible branch November 3, 2026 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant