Skip to content

RS: allow_modules_upload - #4502

Merged
kaitlynmichael merged 3 commits into
mainfrom
DOC-7075
Nov 13, 2026
Merged

kaitlynmichael merged 3 commits into
mainfrom
DOC-7075

Conversation

@kaitlynmichael

@kaitlynmichael kaitlynmichael commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Note

Low Risk
Documentation-only changes with no runtime or API behavior modifications in this diff.

Overview
Documents that custom user-defined module uploads are disabled by default and must be temporarily enabled via the cluster setting allow_modules_upload (using PUT /v1/cluster) before either upload POST endpoint will succeed.

The user-defined modules REST API page now includes a prominent note with links to the cluster object and update request, states that uploads return 403 with modules_upload_disabled while disabled, and lists 403 Forbidden in the status tables for POST /v2/modules/user-defined and POST /v2/local/modules/user-defined/artifacts. Recommended security practices adds a cluster-security bullet to enable uploads only during maintenance and turn the flag off again afterward, because custom modules run as native code on nodes.

Reviewed by Cursor Bugbot for commit 9772ca7. Bugbot is set up for automated code reviews on this repo. Configure here.

@kaitlynmichael
kaitlynmichael requested a review from a team November 6, 2026 18:03
@kaitlynmichael kaitlynmichael self-assigned this Nov 6, 2026
@kaitlynmichael kaitlynmichael added the rs Redis Software label Nov 6, 2026
@CLAassistant

CLAassistant commented Nov 6, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

DOC-7075

@dwdougherty dwdougherty left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just the one comment. LGTM otherwise.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kaitlynmichael
kaitlynmichael merged commit d521e17 into main Nov 13, 2026
57 checks passed
@kaitlynmichael
kaitlynmichael deleted the DOC-7075 branch November 13, 2026 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rs Redis Software

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants