The "Must Encrypt" change will necessarily link into the existing /mail/ web UI so users can read the mail content, so it's probably worth considering possible improvements.

Although I'm suppressing the X-Phabricator-To and X-Phabricator-Cc headers for "Must Encrypt" mail (and the "To: ..." and "Cc: ..." footers in the body), the recipients will still be visible in the actual "To" and "Cc" lines if metamta.one-mail-per-recipient has been configured to false instead of the default (which is true).

On Thread-Topic, this very old thread suggests that Outlook can show the header in some cases: https://bugzilla.mozilla.org/show_bug.cgi?id=411601

@cmmata here is my diff. It can probably be improved, but it works for me:

@mcorteel I have the same problem, arc unit only works with phpunit 5.7. If I use 6.5, it fails. Do you have that patch anywhere with public access so we can use it while it's not published?

Here I was thinking I found a helpful public bug.

From rereading D18600, maybe "ridiculously strong" is an understatement and it was actually so powerful it took effect even if you didn't enable it.

Locally, seems like "No Notifications" correctly has no effect on the "Upload" bubble being shown at HEAD of master.

Hm, actually the problem here is that the notification bubble doesn't show up for any file uploads on our current version. I suppose it could actually be "on our current version + extensions". I'll get a reproduction matrix together tomorrow.

Not unless something super weird/buggy is going on.

We don't have a notifications server, does that make it like that?

It's possible that adjusting Settings → Notifications to No Notifications disables this bubble, if that's a compounding factor here. D18600 pulled back some of the aggressiveness of that setting but I'd guess it might still be hitting the upload progress bubbles.

Ok I actually get a dialog on secure., it's probably because we're out of date with the upstream.

What browser are you using? I get a progress bubble in the lower left immediately. Here's a screenshot -- note the blue upload progress element in the lower left corner.

I think T11934 and an explicit !claim Txxx action (which would mean "reassign + ref"; maybe with a !claim-and-fix Txxx to mean "reassign + fixes" or some kind of !claim+fix X syntax which expands to !claim <args>; !fix <args>) is a possible way forward on this, but I don't want to make "ref" automatically mean "claim".

Because of the high degree of complexity there's basically zero chance we'll ever build this without active customer interest, and we have none today. Closing this until interest arises.

We don't have any use cases for this today so I don't plan to pursue it. It's conceptually reasonable, but complex, and there's no customer interest.

T11934 is a possible way forward on this (and other niche things) without adding a lot of Javascript and UI complexity, by supporting it through !fixes.

One possible approach for this is:

I'm going to roll this forward into T13057.

We shipped "Mentions" and "Duplicates" some time ago.

See T13056 for followup.

One attack is that you can override the content of email and then send invite or welcome mail that says whatever you want, whether the victim has a Phabricator account (welcome) or not (invite).

After T13049 this is relatively within reach, but we haven't seen other requests for it so I don't currently plan to pursue it.

This is relevant now that work related to T13049 has added CSV support.

More or less obsoleted by work in T13049.

That's as far as I plan to take this for now. I think the additional features described above (filter, reorder, reformat, and broader field and application support) are all reasonable if use cases arise, but am not currently aware of use cases. We can return to that stuff when use cases arise.

I just let that run for a while but it finished at some point:

I'm going to roll this forward into T12677 since SES managed to one-up this by a healthy margin in T12237.

(They got back to me and we're scheduling a call.)

Mailgun has yet to respond to me after about three weeks, so I send them a followup.

The compaction completed overnight. I'm optimizing the tables now.

Pool is full again, repo is upgrading, edges are compacting on the instance shard.

web004 is deploying now.

web004 died abruptly so I'm going to fix that and deploy these changes at the same time.

Resolved by D18959 + D18960. See also T13049.

Even on our fairly normal data, the effect was a little bit more dramatic than I'd expected:

D18962 uses this to implement "Export Data" for large result sets.

I'm going to optimize + probe secure001 now and see if any of the tables above shrunk. I'm expecting a very modest effect combined with zero user visible changes in the UI despite throwing away a bunch of data.

Took like 3-ish minutes and did this:

I double checked that our backups are working.

Editing some edges on the new code as a sanity check before I compact things.

(Pushing this to secure, stuff might be funky for a minute while I gently massage the database.)

It looks like PHPExcel was formally deprecated on December 24, 2017 but I'm not planning to switch to its successor, PhpSpreadsheet, until we run into some specific motivator.

See PHI324.

See PHI323.

This hasn't had any stability issues for a while.

transaction.search now returns comments and inlines, including path, line, length, and reply information.

I think everything we know about has been resolved.