FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Disallow running Agents using system user by szykol · Pull Request #753 · splunk/splunk-sdk-python · GitHub

Repository navigation

Disallow running Agents using system user - #753

Merged
szykol merged 1 commit into
developfrom
skolton/system-tokens
Apr 29, 2026
Merged

szykol merged 1 commit into
developfrom
skolton/system-tokens

Conversation

szykol commented Apr 29, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Running agents with system user (splunk-system-user) should not be allowed - such user has permissions to do anything within splunk. Giving this capability to Agent should result in an error being thrown.

Developers should not pass Service objects that use the system tokens to the Agent.

szykol force-pushed the skolton/system-tokens branch 2 times, most recently from 5c7216a to 0fdd2d4 Compare April 29, 2026 07:55
Comment thread splunklib/ai/utils.py Resolved
szykol force-pushed the skolton/system-tokens branch 2 times, most recently from ab92641 to c0cc8cd Compare April 29, 2026 11:27
szykol marked this pull request as ready for review April 29, 2026 11:27
Comment thread splunklib/ai/security.py Resolved
Comment thread splunklib/ai/security.py Resolved
Comment thread tests/unit/ai/test_security.py Resolved
szykol force-pushed the skolton/system-tokens branch from c0cc8cd to 3d4caee Compare April 29, 2026 12:12
szykol force-pushed the skolton/system-tokens branch from 3d4caee to 83092ce Compare April 29, 2026 12:14

mateusz834 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

🚀

szykol merged commit d0197f8 into develop Apr 29, 2026
5 of 7 checks passed
szykol deleted the skolton/system-tokens branch April 29, 2026 12:30
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL