Page MenuHomePhabricator

Tamzin
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Jan 4 2017, 12:46 AM (384 w, 5 d)
Availability
Available
LDAP User
Unknown
MediaWiki User
Tamzin [ Global Accounts ]

Recent Activity

Dec 14 2023

Tamzin added a comment to T352827: Directory traversal allows single-page whitelisting to override entire spam-blacklist entry.

@Beetstra The ../ vector works on all websites, if I'm understanding T352827#9391030 correctly, while URL-parameter-based tricks like ?realPage=BadPage will depend on the website in question. The way it seems to me, someone whitelisting a page or path is expecting to whitelist just that page or path. But the use of \b—which makes sense on the blacklist, where we want to match the whole site—has the opposite effect on the whitelist. So most of these URLs should have been using $ to begin with.

Dec 14 2023, 8:35 PM · SecTeam-Processed, Vuln-Misconfiguration, SpamBlacklist, Security, Security-Team

Dec 9 2023

Tamzin added a comment to T352827: Directory traversal allows single-page whitelisting to override entire spam-blacklist entry.

That makes a lot of sense, @matmarex, thanks. What about the URL parameters angle, though? To give a real example that seems ripe for exploitation, on enwiki the neo-Nazi site Metapedia is blacklisted, but \ben\.metapedia\.org/wiki/Main_Page\b is whitelisted. That allows a link like https://en.metapedia.org/wiki/Main_Page?oldid=479369, which in fact would lead readers to Holocaust denial.

Dec 9 2023, 6:54 AM · SecTeam-Processed, Vuln-Misconfiguration, SpamBlacklist, Security, Security-Team

Dec 7 2023

Tamzin added a comment to T352827: Directory traversal allows single-page whitelisting to override entire spam-blacklist entry.

Oh, postscript to my original post: Is the \S in my proposed regex necessary, or does the extension implicitly stop looking for a match when it hits whitespace? If the latter, a simple . would work.

Dec 7 2023, 3:51 PM · SecTeam-Processed, Vuln-Misconfiguration, SpamBlacklist, Security, Security-Team
Tamzin updated subscribers of T352827: Directory traversal allows single-page whitelisting to override entire spam-blacklist entry.

I want to be clear, I hesitate to even call this a bug or a misconfiguration on the extension's end; it's just counterintuitive behavior for end users. With https://badsite.com/page/../../badpage, for instance, the extension is correctly saying "This does match \bbadsite\.com\b, so it should normally be blocked, but it also matches \bbadsite\.com/page\b, so it should be let through". That's all valid parsing of the regexen. The problem lies in how URLs actually work (or, tend to work on most websites), which is beyond the purview of a simple regex-based extension. (If this were, say, a profanity-matching extension, it would be working perfectly.)

Dec 7 2023, 3:27 PM · SecTeam-Processed, Vuln-Misconfiguration, SpamBlacklist, Security, Security-Team

Dec 6 2023

Tamzin added a project to T352827: Directory traversal allows single-page whitelisting to override entire spam-blacklist entry: SpamBlacklist.
Dec 6 2023, 1:45 AM · SecTeam-Processed, Vuln-Misconfiguration, SpamBlacklist, Security, Security-Team
Tamzin updated subscribers of T352827: Directory traversal allows single-page whitelisting to override entire spam-blacklist entry.
Dec 6 2023, 1:44 AM · SecTeam-Processed, Vuln-Misconfiguration, SpamBlacklist, Security, Security-Team
Tamzin created T352827: Directory traversal allows single-page whitelisting to override entire spam-blacklist entry.
Dec 6 2023, 1:43 AM · SecTeam-Processed, Vuln-Misconfiguration, SpamBlacklist, Security, Security-Team

Aug 2 2023

Tamzin added a comment to T303433: Allow Stewards to enable 'emergency CAPTCHAs' for anonymous IP edits.

This is all great long-term planning (seriously!) but is there a chance of making a stopgap, light-weight solution for just EmergencyCaptcha? Even if it's as simple as, I don't know, a page editable only by stewards (a few ways to implement that) that allows for a comma-separated list of wikis to have EmergencyCaptcha on on; have a script check it every 5 minutes or something. (Probably not ideal implementation, but just giving an idea of a fairly cheap implementation.)

Aug 2 2023, 3:57 PM · MediaWiki-Platform-Team (Radar), MW-1.39-notes (1.39.0-wmf.25; 2022-08-15), Stewards-and-global-tools, MediaWiki-extensions-CentralAuth, SecTeam-Processed, Sustainability (Incident Followup), ConfirmEdit (CAPTCHA extension), Platform Engineering, Wikimedia-Site-requests, Security

Jul 29 2023

Tamzin created T343082: Race condition between 2 single-revision revdels can cause unintentional unhiding.
Jul 29 2023, 11:12 PM · MediaWiki-Revision-deletion

Jul 12 2023

Tamzin added a comment to T275155: Homepage: allow users to remove the startemail module from the homepage once their email is confirmed.

I think the issue is, unless I'm misunderstanding, the user only gets the toast message if they've already clicked the X. So someone who thinks the X is to remove their email will never get the message saying that it isn't. I think removing by default is a good thing to consider. I'd think basically all users in 2023 have an expectation that any website they verify their email with still knows that email, without needing a prominent reminder.

Jul 12 2023, 9:27 PM · Security, Growth-Team, GrowthExperiments-StartModule, GrowthExperiments-Homepage
Tamzin added a comment to T341195: Clarify in GrowthExperiments Homepage that email addresses are not public.

@KStoller-WMF Hmm my only concern there, if we're thinking about the lowest common denominator, is that some users might think that the "X" is to remove their email from the account. I think the solution works in principle, though.

Jul 12 2023, 8:09 PM · Growth-Team, GrowthExperiments-Homepage

Jul 6 2023

Tamzin created T341195: Clarify in GrowthExperiments Homepage that email addresses are not public.
Jul 6 2023, 10:38 AM · Growth-Team, GrowthExperiments-Homepage

Apr 5 2023

Tamzin added a comment to T333723: w.wiki + ?withJS= allows an intadmin on any wiki to launch phishing attacks on all wikis, or lets any user trick people into running unwanted JS.

Is there something I'm missing?

Apr 5 2023, 6:15 AM · Vuln-Misconfiguration, MediaWiki-extensions-UrlShortener, Security, Security-Team

Mar 31 2023

Tamzin created T333723: w.wiki + ?withJS= allows an intadmin on any wiki to launch phishing attacks on all wikis, or lets any user trick people into running unwanted JS.
Mar 31 2023, 8:09 PM · Vuln-Misconfiguration, MediaWiki-extensions-UrlShortener, Security, Security-Team

Jan 24 2023

Tamzin added a comment to T327815: Repeated loss of session data on edit attempt.

Anecdotal reports from several users of concurrent issues logging in.

Jan 24 2023, 9:03 PM · Wikimedia-Incident, Wikimedia-production-error, MediaWiki-Core-AuthManager

Oct 2 2022

Tamzin added a comment to T318746: Use systemd to autorestart Celery workers.

Meanwhile over here:

Oct 2 2022, 12:59 AM · Community-Tech (CommTech-Kanban), WikiWho
Tamzin added a comment to T318746: Use systemd to autorestart Celery workers.

Persisting here for a few days now (Chrome on ChromeOS). Going to boldly set this as "Unbreak Now!" since the extension is currently unusable for many/all people.

Oct 2 2022, 12:26 AM · Community-Tech (CommTech-Kanban), WikiWho
Tamzin triaged T318746: Use systemd to autorestart Celery workers as Unbreak Now! priority.
Oct 2 2022, 12:24 AM · Community-Tech (CommTech-Kanban), WikiWho

Jul 11 2022

Tamzin added a comment to T312815: DBPerformance: Expectation readQueryTime <= 5 not met (from ApiQueryUserContribs).

I think this is due to one of the three users I had in the query being a high-edit-count user (>300k). That would make sense if the DB is applying the limit only after running the whole query, which I gather is how it works? For the purposes of what I'm doing, I can just exclude users with >100k edits, and that ought to handle it. But there's some use cases at least where that wouldn't be an option.

Jul 11 2022, 11:15 PM · MediaWiki-Action-API, Wikimedia-Slow-DB-Query, Wikimedia-production-error

Jun 6 2022

Tamzin created T310015: Special:Nuke output parses [[:$1]] for a file as a link to $'"1 (that's single quote, then double).
Jun 6 2022, 6:49 PM · MW-1.42-notes (1.42.0-wmf.9; 2023-12-12), MediaWiki-extensions-Nuke

May 27 2022

Tamzin added a comment to T309366: Investigate: IP info not shown when all IP's edits are deleted.

Since there will be some cases where no relevant log entries exist either, e.g. https://en.wikipedia.org/wiki/Special:Contributions/2803:D100:E080:306:787A:3962:E1E7:52EB, would there be any way to work from deleted contribs, at least for admins? Otherwise we have a situation where an LTA could cause a lot of disruption on a deletion-eligible page that someone else created, that page gets deleted, and now there's no way to use IP Info on the IP the LTA was using, short of undeleting a revision, which is something of a perverse incentive.

May 27 2022, 1:08 AM · Anti-Harassment (AHT Sprint 17: The Fruit Hat), IP Info

Apr 15 2022

Tamzin added a comment to T306232: "Download as PDF" on an old revision goes to Special:Book rather than Special:DownloadAsPdf.

@Aklapper Sorry, I thought these steps were clear enough. The reason I think this is a bug is a) that it is inconsistent behavior and b) that there is thus no obvious way to download a PDF of an old revision (if such a thing is possible).

("Printable version" works as expected in both cases.)

Apr 15 2022, 7:16 PM · Electron-PDFs, Browser-support-print-media
Tamzin updated the task description for T306232: "Download as PDF" on an old revision goes to Special:Book rather than Special:DownloadAsPdf.
Apr 15 2022, 2:36 AM · Electron-PDFs, Browser-support-print-media
Tamzin renamed T306232: "Download as PDF" on an old revision goes to Special:Book rather than Special:DownloadAsPdf from "Download as PDF" on an old revision goes to [[Special:Book]] rather than Special:DownloadAsPdf to "Download as PDF" on an old revision goes to Special:Book rather than Special:DownloadAsPdf.
Apr 15 2022, 2:32 AM · Electron-PDFs, Browser-support-print-media
Tamzin created T306232: "Download as PDF" on an old revision goes to Special:Book rather than Special:DownloadAsPdf.
Apr 15 2022, 2:32 AM · Electron-PDFs, Browser-support-print-media

Mar 31 2022

Tamzin added a comment to T305008: Forcibly creating a local account causes autoblocks for the user to affect the creating administrator's IP address.

I happen to have force-created an account on testwiki a while ago for T302771, so I just tried blocking it there. An autoblock was made, but there's no block notice if I go to edit logged-out... I'm guessing the difference is my IP has probably changed since I force-created the account a month ago, and by blocking the "last IP address" used by the account I force-created, it's really just blocking whatever IP I had the day I created it?

Mar 31 2022, 6:21 AM · CheckUser, MediaWiki-extensions-CentralAuth

Mar 29 2022

Tamzin added a comment to T305011: Simple regular expression fails on 10000 character string.

My first thought was something about capturing groups, but it also fails on (?:x)*. I then played guess-and-check with quantifiers, and found that it works fine up till (x){0,2729}, but anything past that fails. Hmm.

Mar 29 2022, 10:39 PM · AbuseFilter

Mar 27 2022

Tamzin updated the task description for T304777: Special:Nuke's output creates external links for URLs in page titles.
Mar 27 2022, 12:32 AM · User-DannyS712, MW-1.39-notes (1.39.0-wmf.6; 2022-04-04), MediaWiki-extensions-Nuke
Tamzin updated the task description for T304777: Special:Nuke's output creates external links for URLs in page titles.
Mar 27 2022, 12:31 AM · User-DannyS712, MW-1.39-notes (1.39.0-wmf.6; 2022-04-04), MediaWiki-extensions-Nuke
Tamzin created T304777: Special:Nuke's output creates external links for URLs in page titles.
Mar 27 2022, 12:27 AM · User-DannyS712, MW-1.39-notes (1.39.0-wmf.6; 2022-04-04), MediaWiki-extensions-Nuke

Mar 18 2022

Tamzin updated subscribers of T304171: Breaking change in parse API: level was string, is now int.

See current behavior with int level values at this API result. Contrast with an apparent cached result that has str level values here (picking an arbitrary page unlikely to be edited anytime soon).

Mar 18 2022, 3:52 PM · MW-1.38-notes, MW-1.39-notes (1.39.0-wmf.1; 2022-03-21-early), MW-1.38-release, MediaWiki-Parser, MediaWiki-Action-API

Mar 8 2022

Tamzin created T303337: Flow should not require a summary when deleting a topic.
Mar 8 2022, 10:18 PM · StructuredDiscussions, Growth-Team

Mar 2 2022

Tamzin awarded T298638: Make modifications to Pager HTML to add heading separators to support Minerva skinning a Dislike token.
Mar 2 2022, 5:22 PM · User-notice-archive, User-brennen, MW-1.38-notes (1.38.0-wmf.24; 2022-02-28), Patch-For-Review, Platform Engineering, Web-Team-Backlog (Kanbanana-FY-2021-22), MediaWiki-Page-history, User-Jdlrobson, MobileFrontend (MobileFrontend Special Pages)
Tamzin added a comment to T298638: Make modifications to Pager HTML to add heading separators to support Minerva skinning.

This should not have been rolled out without discussion of accessibility, especially since breaking up an HTML list is a classic a11y concern. It should be reverted until an accessible version can be rolled out, rather than left in production while a fix is worked on. I don't think we'd be okay with a comparably aggrevating change (based on what @Graham87 has said) staying in prod if it had comparably disruptive changes for non–screenreader users. If that accessible version can also not break list numbering (a tool I find invaluable in my workflow on-wiki... and yes, yes, https://xkcd.com/1172/, I know), that would be a great added benefit, but my primary concern is a11y.

Mar 2 2022, 3:26 PM · User-notice-archive, User-brennen, MW-1.38-notes (1.38.0-wmf.24; 2022-02-28), Patch-For-Review, Platform Engineering, Web-Team-Backlog (Kanbanana-FY-2021-22), MediaWiki-Page-history, User-Jdlrobson, MobileFrontend (MobileFrontend Special Pages)

Mar 1 2022

Tamzin added a comment to T302771: Local accounts forcibly created with Special:CreateLocalAccount should have a unique method in Special:CentralAuth.

I concur that this should be noted separately. It's an unusual circumstance that merits noting in the CentralAuth; it goes against the expectation that an account being attached on a wiki means that the user has visited it or interacted with it in some way; and, most of all, it's simply incorrect to say "created on login" for an account that, in the case of the example given above, I can tell you I have not logged in to since 2013.

Mar 1 2022, 6:44 AM · MediaWiki-extensions-CentralAuth

Jan 15 2022

Tamzin added a comment to T293321: Special:AbuseLog uses `mw-anonuserlink` for unattached accounts.

@Daimona I found it in one other place. For whatever reason, there are a few blocks in Special:BlockList on enwiki logged against nonexistent users. (Not sure if that needs to be a bug of its own, or already is one.) This same thing happens there. See e.g. James5smith and SLR Consulting Ltd @ this BlockList query.

Jan 15 2022, 12:03 PM · AbuseFilter, CSS

Oct 14 2021

Tamzin updated the task description for T293321: Special:AbuseLog uses `mw-anonuserlink` for unattached accounts.
Oct 14 2021, 12:12 AM · AbuseFilter, CSS
Tamzin triaged T293321: Special:AbuseLog uses `mw-anonuserlink` for unattached accounts as Lowest priority.
Oct 14 2021, 12:11 AM · AbuseFilter, CSS
Tamzin created T293321: Special:AbuseLog uses `mw-anonuserlink` for unattached accounts.
Oct 14 2021, 12:06 AM · AbuseFilter, CSS

Oct 5 2021

Tamzin added a comment to T219279: Some pages will become completely unreachable after PHP7 update due to Unicode changes.

Ah. That's a shame. You'd think after two renames I'd know that. Oh well. If anyone does try to awaken a dormant account, they'll probably just think they forgot the password.

Oct 5 2021, 5:42 PM · User-notice-archive, MW-1.38-notes (1.38.0-wmf.6; 2021-10-26), Patch-For-Review, MW-1.35-notes (1.35.0-wmf.28; 2020-04-14), Platform Team Workboards (Clinic Duty Team), MW-1.34-notes (1.34.0-wmf.16; 2019-07-30), serviceops, SRE, PHP 7.2 support, MediaWiki-General
Tamzin added a comment to T219279: Some pages will become completely unreachable after PHP7 update due to Unicode changes.

with usertalk notes pointing them to Special:GlobalRenameRequest?

That's the issue, I can only leave a note on User_Talk:<capitalized username> (technical rename) page, not User_Talk:<unreachable_lowercase_username> since that user_talk will also be unreachable. And the capitalized version belongs to a different user. And without a notice if the owner of the account decides to come back, how would they know where to look for the note..

Oct 5 2021, 5:32 PM · User-notice-archive, MW-1.38-notes (1.38.0-wmf.6; 2021-10-26), Patch-For-Review, MW-1.35-notes (1.35.0-wmf.28; 2020-04-14), Platform Team Workboards (Clinic Duty Team), MW-1.34-notes (1.34.0-wmf.16; 2019-07-30), serviceops, SRE, PHP 7.2 support, MediaWiki-General
Tamzin added a comment to T219279: Some pages will become completely unreachable after PHP7 update due to Unicode changes.

Was just about to ask this. While rare, there are cases of accounts making their first edits years after creation. Maybe systematically rename them to something like <capitalized username> (technical rename), with usertalk notes pointing them to Special:GlobalRenameRequest?

Oct 5 2021, 5:19 PM · User-notice-archive, MW-1.38-notes (1.38.0-wmf.6; 2021-10-26), Patch-For-Review, MW-1.35-notes (1.35.0-wmf.28; 2020-04-14), Platform Team Workboards (Clinic Duty Team), MW-1.34-notes (1.34.0-wmf.16; 2019-07-30), serviceops, SRE, PHP 7.2 support, MediaWiki-General

May 13 2021

Tamzin added a comment to T282748: Rename Phabricator account PinkAmpersand to Tamzin.

Thanks! :)

May 13 2021, 4:20 PM · Phabricator
Aklapper renamed Tamzin from PinkAmpersand to Tamzin.
May 13 2021, 11:19 AM

May 12 2021

Tamzin created T282748: Rename Phabricator account PinkAmpersand to Tamzin.
May 12 2021, 8:46 PM · Phabricator
Tamzin added a comment to T66778: Echo: Remove the signature-requirement for Mention notifications.

Idea: __PING__ and __NOPING__ magic words. Talk pages would be __PING__ by default. Content pages would be __NOPING__ by default. Wikis would typically add __NOPING__ to all archive templates. There could be safeguards against false positives, like "Ignore edits with multiple section headings" or "Ignore edits that start and end on different indentation levels." When an edit is ignored for one of these reasons, notify the user that no ping was sent (rather than the status quo, where you only get a "failed notification" message if the notification was sendable but not delivered.) That way false negatives aren't that big of a deal. And maybe have a way for users to intentionally disable pings for an edit, like by including "NOPING!" in their edit summary.

May 12 2021, 4:29 PM · Growth-Team-Filtering, Growth-Team, Notifications

Sep 5 2017

Tamzin added a comment to T174977: {{REVISIONID}} returns null, causing templates to be rendered in Preview mode (when using REST API).

Since this is affecting the display of a large number of en.wp pages, I've boldly set it as high priority. My apologies if I shouldn't have.

Sep 5 2017, 12:08 AM · Patch-For-Review, Mobile-Content-Service, Product-Infrastructure-Team-Backlog-Deprecated, Parsoid, Mobile, Wikipedia-Android-App-Backlog, Android-app-Bugs
Tamzin triaged T174977: {{REVISIONID}} returns null, causing templates to be rendered in Preview mode (when using REST API) as High priority.
Sep 5 2017, 12:08 AM · Patch-For-Review, Mobile-Content-Service, Product-Infrastructure-Team-Backlog-Deprecated, Parsoid, Mobile, Wikipedia-Android-App-Backlog, Android-app-Bugs
Tamzin created T174977: {{REVISIONID}} returns null, causing templates to be rendered in Preview mode (when using REST API).
Sep 5 2017, 12:07 AM · Patch-For-Review, Mobile-Content-Service, Product-Infrastructure-Team-Backlog-Deprecated, Parsoid, Mobile, Wikipedia-Android-App-Backlog, Android-app-Bugs

Sep 4 2017

Tamzin added a comment to T170726: Template-protected and ECP pages being improperly treated as fully protected.

@Dbrant Yep, it's working fine now. Thanks. Although I did get an error when I tried to edit a Lua module on the app. Just a generic "an error occurred," not the "page protected" message I was running into here. Is that a known bug(/feature?), or should I report it?

Sep 4 2017, 11:40 PM · Wikipedia-Android-App-Backlog (Android Release FY2020-21), Android-app-Bugs

Jul 14 2017

Tamzin created T170726: Template-protected and ECP pages being improperly treated as fully protected.
Jul 14 2017, 10:40 PM · Wikipedia-Android-App-Backlog (Android Release FY2020-21), Android-app-Bugs