Skip to content

Member Content: Tags: Require Login#1136

Draft
n7studios wants to merge 4 commits into
mainfrom
restrict-content-tags-require-login
Draft

Member Content: Tags: Require Login#1136
n7studios wants to merge 4 commits into
mainfrom
restrict-content-tags-require-login

Conversation

@n7studios

@n7studios n7studios commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes this reported vulnerability by requiring that Member Content gated by Tag enforce the login / magic link method to obtain a signed subscriber ID to access the content gated by tag, in the same way the Plugin functions for content gated by Kit Form or Kit Product.

As a result, numeric, non signed subscriber IDs cannot access gated content by tag, which mirrors the behaviour for gated content by Kit Form and Kit Product.

Slack thread.

Testing

Updated tests to reflect that gating content by tag requires a login.

Checklist

@n7studios n7studios self-assigned this Jul 15, 2026
@n7studios n7studios added the bug label Jul 15, 2026

@ciccio-kit ciccio-kit left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants