Skip to content

Member Content: Tags: Require Login#1136

Merged
n7studios merged 8 commits into
fix-sequence-testsfrom
restrict-content-tags-require-login
Jul 17, 2026
Merged

Member Content: Tags: Require Login#1136
n7studios merged 8 commits into
fix-sequence-testsfrom
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 ✅

@github-actions

Copy link
Copy Markdown

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios changed the base branch from main to fix-sequence-tests July 16, 2026 08:38
@n7studios n7studios marked this pull request as ready for review July 16, 2026 08:38
…-in-cookie

Remove Store Subscriber Email as ID
@n7studios n7studios merged commit 6178a2c into fix-sequence-tests Jul 17, 2026
68 of 96 checks passed
@n7studios n7studios deleted the restrict-content-tags-require-login branch July 17, 2026 00:39
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