Page MenuHomePhabricator

Prepare Newsletter extension for IP Masking
Closed, ResolvedPublic

Description

A preliminary investigation (T326759) has found that the Newsletter extension may be affected by IP Masking

Event Timeline

Duplicate of T344718? Is there something here other than requireLogin calls?

kostajh claimed this task.
kostajh added subscribers: STran, Dreamy_Jazz, kostajh.

Duplicate of T344718? Is there something here other than requireLogin calls?

Yes, this task seems to be a duplicate of T344718: Update requireLogin() for temp users on Newsletter extension. I agree with the comment in T344718#9111599:

I see no specific reason why temporary users shouldn't be allowed to create Newsletters, if the wiki config gives them the newsletter-create right. Likewise per T330509: [IP Masking] Make Echo Notifications available to temporary users temp users can receive notifications, so they should be allowed to subscribe as well. Proposing to close without action.

As such I'm going to close this. (cc @STran @Dreamy_Jazz)