8000 fix: restrict synthetic events to administrators by sielicki · Pull Request #1901 · slskd/slskd · GitHub
[go: up one dir, main page]

Skip to content

fix: restrict synthetic events to administrators - #1901

Closed
sielicki wants to merge 1 commit into
slskd:masterfrom
sielicki:fix/restrict-synthetic-events
Closed

sielicki wants to merge 1 commit into
slskd:masterfrom
sielicki:fix/restrict-synthetic-events

Conversation

@sielicki
Copy link
Copy Markdown

Summary

Require administrator access to raise synthetic events, which can trigger configured scripts and webhooks.

Testing

The .NET build and 1,490 unit tests passed via Nix. Changed C# files passed formatting checks.

Synthetic events can trigger configured scripts and webhooks.
Require administrator access and cover the role check with a test.

Signed-off-by: Nicholas Sielicki <opensource@nslick.com>
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.

2 participants

0