Page MenuHomePhabricator
Feed Advanced Search

Jun 19 2019

epriestley added a comment to T13319: Transactions which modify EditEngine form default values don't include details about the old and new values.

There are a few pieces here, since we have to thread the needle through many layers to get where we want to go.

Jun 19 2019, 8:16 PM · EditEngine
epriestley created T13319: Transactions which modify EditEngine form default values don't include details about the old and new values.
Jun 19 2019, 7:54 PM · EditEngine
epriestley closed T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility) as Resolved by committing Restricted Diffusion Commit.
Jun 19 2019, 7:51 PM · Mail, Phacility
epriestley added a comment to T4411: Adding a CC to a Maniphest Task should give View rights for that user.

See also T13317.

Jun 19 2019, 6:20 PM · Wikimedia, Policy
epriestley added a revision to T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility): Unknown Object (Differential Revision).
Jun 19 2019, 6:15 PM · Mail, Phacility
epriestley added a revision to T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion: Unknown Object (Differential Revision).
Jun 19 2019, 5:46 PM · Remarkup, Diffusion
epriestley added a revision to T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion: Restricted Differential Revision.
Jun 19 2019, 5:44 PM · Remarkup, Diffusion
epriestley added a comment to T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion.

See PHI1312. The modified JIRA rules reportedly misfire on URIs with anchors or parameters. For now, the plan is to just skip these URIs. In the future, we might specialize URIs which (for example) link to comments on foreign objects.

Jun 19 2019, 5:33 PM · Remarkup, Diffusion
epriestley closed T9720: Provide a way to link to Diffusion files, a subtask of T5378: Phabricator full task URLs for that instance do not provide hover/popup text (like T1234 does), as Resolved.
Jun 19 2019, 5:32 PM · Customer Impact, Restricted Project, Restricted Project, Hovercards, Remarkup, Wikimedia
epriestley added a comment to T9720: Provide a way to link to Diffusion files.

I'm going to close this in favor of T13291. Referencing files by full URI currently works, and relative references and other behavior is planned.

Jun 19 2019, 5:32 PM · Restricted Project, Remarkup
epriestley closed T9720: Provide a way to link to Diffusion files, a subtask of T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion, as Resolved.
Jun 19 2019, 5:32 PM · Remarkup, Diffusion
epriestley added a parent task for T9720: Provide a way to link to Diffusion files: T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion.
Jun 19 2019, 5:31 PM · Restricted Project, Remarkup
epriestley added a subtask for T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion: T9720: Provide a way to link to Diffusion files.
Jun 19 2019, 5:31 PM · Remarkup, Diffusion
epriestley added a comment to T13305: Migration "20180208.maniphest.02.populate.php" may fail when queueing indexing tasks.

Getting rid of the indirect writes on handle reads would probably be nice eventually, but doesn't directly accomplish anything today.

Jun 19 2019, 5:30 PM · Installing & Upgrading, Database
epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

I think that might be everything? Not entirely sure, but haven't seen any more since the last deploy.

Jun 19 2019, 5:29 PM · Transactions
epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Unknown Object (Differential Revision).
Jun 19 2019, 5:29 PM · Transactions
epriestley added a revision to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it: Unknown Object (Differential Revision).
Jun 19 2019, 5:24 PM · Quicksand
epriestley added a comment to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.

D20573 has one issue where the "Select" buttons in the "Merge Duplicates In" dialog are now closing the dialog. We might be missing a JX.pass() / e.prevent() (hopefully) or might need to get slightly more creative in distinguishing between navigation links and javascript magic links.

Jun 19 2019, 2:51 PM · Quicksand

Jun 18 2019

epriestley closed T13308: Phacility account import flow can misfire when both invites and account links are present as Resolved by committing Restricted Diffusion Commit.
Jun 18 2019, 10:49 PM · Phacility
epriestley closed T13146: Phacility account import tool fails to recognize usernames with differing case, e.g. "Alice" vs "alice" as Resolved by committing Restricted Diffusion Commit.
Jun 18 2019, 10:31 PM · Phacility
epriestley closed T13312: "Branch Status" in Diffusion has a bizarre pager at the top of the element as Resolved by committing Restricted Diffusion Commit.
Jun 18 2019, 10:20 PM · Diffusion
epriestley closed T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone as Resolved by committing Restricted Diffusion Commit.
Jun 18 2019, 10:15 PM · Diffusion
epriestley closed T13304: Phabricator "Backups" instructions are somewhat out of date as Resolved by committing Restricted Diffusion Commit.
Jun 18 2019, 6:37 PM · Database
epriestley closed T13315: "bin/repository reparse" stops when encountering a "PhabricatorWorkerPermanentFailureException", but should continue as Resolved by committing Restricted Diffusion Commit.
Jun 18 2019, 6:33 PM · Diffusion
epriestley updated subscribers of T13156: Plans: Improve Phacility UI for managing instance managers and cards.

See PHI954. The "support pact members" cache is currently updated only when you actually click "Support", but this is often silly and unintuitive.

Jun 18 2019, 6:22 PM · Plans, Phacility
epriestley created T13318: Phacility developer setup documentation omits "local.*" hosts.
Jun 18 2019, 6:20 PM · Phacility
epriestley renamed T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility) from Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility to Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).
Jun 18 2019, 5:22 PM · Mail, Phacility
epriestley added a comment to T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).

Aha! This user very cleverly added noreply@admin.phacility.com to their user account before they were disabled.

Jun 18 2019, 5:22 PM · Mail, Phacility
epriestley created T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).
Jun 18 2019, 5:15 PM · Mail, Phacility
epriestley created T13316: On Workboards, "Move Tasks to Column..." has high friction in the common use case (selecting a column on the same board).
Jun 18 2019, 4:41 PM · Workboards
epriestley added a revision to T13315: "bin/repository reparse" stops when encountering a "PhabricatorWorkerPermanentFailureException", but should continue: Unknown Object (Differential Revision).
Jun 18 2019, 3:02 PM · Diffusion
epriestley created T13315: "bin/repository reparse" stops when encountering a "PhabricatorWorkerPermanentFailureException", but should continue.
Jun 18 2019, 2:57 PM · Diffusion

Jun 17 2019

epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

I deployed that last round of things to secure. Not totally confident I got everything, but hopefully we're in better shape now.

Jun 17 2019, 9:25 PM · Transactions
epriestley closed T13311: Audit code does not fully respect committer identities as commit authors as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 8:56 PM · Diffusion
epriestley closed T13310: Users with no access to Spaces can not log out as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 8:44 PM · Auth, Spaces
epriestley closed T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 8:31 PM · Differential, Files
epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Unknown Object (Differential Revision).
Jun 17 2019, 8:11 PM · Transactions
epriestley added a comment to T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled.

The change above seems to have worked around things, I'll leave this open until we can confirm that PHP 7.3.6 or some later version fixes the underlying issue.

Jun 17 2019, 6:02 PM · Infrastructure
epriestley added a revision to T13312: "Branch Status" in Diffusion has a bizarre pager at the top of the element: Unknown Object (Differential Revision).
Jun 17 2019, 5:52 PM · Diffusion
epriestley closed T13307: "Sign With MFA" requires "CAN_EDIT" permission, but should only require "CAN_INTERACT" permission as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 5:41 PM · Auth
epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Unknown Object (Differential Revision).
Jun 17 2019, 5:40 PM · Transactions
epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Unknown Object (Differential Revision).
Jun 17 2019, 5:36 PM · Transactions
epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

uhoh looks like I'm a dum dum

Jun 17 2019, 5:33 PM · Transactions
epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

This is doing somewhat better now, but I've still seen:

Jun 17 2019, 5:23 PM · Transactions
epriestley added a revision to T13311: Audit code does not fully respect committer identities as commit authors: Unknown Object (Differential Revision).
Jun 17 2019, 5:11 PM · Diffusion
epriestley added a revision to T13311: Audit code does not fully respect committer identities as commit authors: Unknown Object (Differential Revision).
Jun 17 2019, 5:04 PM · Diffusion
epriestley added a revision to T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB: Unknown Object (Differential Revision).
Jun 17 2019, 4:51 PM · Differential, Files
epriestley created T13314: Poke around `bin/repository update` and chase off any ghosts.
Jun 17 2019, 4:06 PM · Diffusion
epriestley created T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB.
Jun 17 2019, 2:40 PM · Differential, Files
epriestley added a comment to T13312: "Branch Status" in Diffusion has a bizarre pager at the top of the element.

If this table is retained, there's a mystery meat "branch" icon meaning "default branch" in the leftmost column. This should get a label/tip.

Jun 17 2019, 2:31 PM · Diffusion
epriestley created T13312: "Branch Status" in Diffusion has a bizarre pager at the top of the element.
Jun 17 2019, 2:30 PM · Diffusion

Jun 14 2019

epriestley created T13311: Audit code does not fully respect committer identities as commit authors.
Jun 14 2019, 4:02 PM · Diffusion

Jun 13 2019

epriestley added a revision to T13310: Users with no access to Spaces can not log out: Restricted Differential Revision.
Jun 13 2019, 11:59 PM · Auth, Spaces
epriestley created T13310: Users with no access to Spaces can not log out.
Jun 13 2019, 11:49 PM · Auth, Spaces
epriestley added a revision to T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone: Unknown Object (Differential Revision).
Jun 13 2019, 11:31 PM · Diffusion
epriestley added a comment to T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone.

The shallow <hash> frames can not advertise capabilities.

Jun 13 2019, 11:28 PM · Diffusion
epriestley added a comment to T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone.

Here's what goes over the wire:

Jun 13 2019, 10:46 PM · Diffusion
epriestley added a comment to T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone.

To reproduce this, you can:

Jun 13 2019, 10:42 PM · Diffusion
epriestley created T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone.
Jun 13 2019, 10:42 PM · Diffusion
epriestley added a revision to T13308: Phacility account import flow can misfire when both invites and account links are present: Unknown Object (Differential Revision).
Jun 13 2019, 10:15 PM · Phacility
epriestley added a comment to T13308: Phacility account import flow can misfire when both invites and account links are present.

To reproduce the first issue, do this:

Jun 13 2019, 8:43 PM · Phacility
epriestley created T13308: Phacility account import flow can misfire when both invites and account links are present.
Jun 13 2019, 8:23 PM · Phacility
epriestley added a revision to T13146: Phacility account import tool fails to recognize usernames with differing case, e.g. "Alice" vs "alice": Unknown Object (Differential Revision).
Jun 13 2019, 8:10 PM · Phacility

Jun 11 2019

epriestley updated the task description for T12664: Update diff/patch parsing to extract more metadata and parse a wider range of formats.
Jun 11 2019, 6:18 PM · Arcanist, Differential

Jun 6 2019

epriestley added a project to T13307: "Sign With MFA" requires "CAN_EDIT" permission, but should only require "CAN_INTERACT" permission: Auth.
Jun 6 2019, 2:23 AM · Auth
epriestley added a revision to T13307: "Sign With MFA" requires "CAN_EDIT" permission, but should only require "CAN_INTERACT" permission: Unknown Object (Differential Revision).
Jun 6 2019, 2:23 AM · Auth
epriestley created T13307: "Sign With MFA" requires "CAN_EDIT" permission, but should only require "CAN_INTERACT" permission.
Jun 6 2019, 2:18 AM · Auth
epriestley added a subtask for T10469: Implement "Tabs", an amazing feature where multiple different pages are loaded in the same browser window!: T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.
Jun 6 2019, 1:58 AM · Quicksand, Feature Request, Workboards
epriestley added a parent task for T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it: T10469: Implement "Tabs", an amazing feature where multiple different pages are loaded in the same browser window!.
Jun 6 2019, 1:58 AM · Quicksand
epriestley renamed T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it from With Quicksand enabled, "Choose Subtype" dialog on "Create Subtask" workflow is not dismissed by making a selection to When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.
Jun 6 2019, 1:57 AM · Quicksand
epriestley added a comment to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.

D20573 will fix this, but perhaps not in the most graceful way possible.

Jun 6 2019, 1:56 AM · Quicksand
epriestley added a revision to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it: Unknown Object (Differential Revision).
Jun 6 2019, 1:50 AM · Quicksand

Jun 5 2019

epriestley added a revision to T13304: Phabricator "Backups" instructions are somewhat out of date: Unknown Object (Differential Revision).
Jun 5 2019, 10:00 PM · Database
epriestley added a revision to T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster: Unknown Object (Differential Revision).
Jun 5 2019, 9:36 PM · Phacility, Files
epriestley added a revision to T13305: Migration "20180208.maniphest.02.populate.php" may fail when queueing indexing tasks: Unknown Object (Differential Revision).
Jun 5 2019, 6:43 PM · Installing & Upgrading, Database
epriestley claimed T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster.
Jun 5 2019, 5:59 PM · Phacility, Files
epriestley created T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster.
Jun 5 2019, 5:59 PM · Phacility, Files
epriestley added a comment to T13305: Migration "20180208.maniphest.02.populate.php" may fail when queueing indexing tasks.

I applied this change to 20180208.maniphest.02.populate.php:

Jun 5 2019, 5:24 PM · Installing & Upgrading, Database
epriestley created T13305: Migration "20180208.maniphest.02.populate.php" may fail when queueing indexing tasks.
Jun 5 2019, 5:18 PM · Installing & Upgrading, Database
epriestley created T13304: Phabricator "Backups" instructions are somewhat out of date.
Jun 5 2019, 3:26 PM · Database

Jun 4 2019

epriestley merged T10307: Option to move child pages when parent in renamed into T6004: When moving a wiki page with subpages prompt to also move subpages.
Jun 4 2019, 8:01 PM · Phriction
epriestley merged task T10307: Option to move child pages when parent in renamed into T6004: When moving a wiki page with subpages prompt to also move subpages.
Jun 4 2019, 8:01 PM · Feature Request, Phriction
epriestley added a comment to T6004: When moving a wiki page with subpages prompt to also move subpages.

See also PHI1275. Now that Phriction supports relative references in [[ ... ]] syntax, moves should be able to modify those references.

Jun 4 2019, 7:46 PM · Phriction
epriestley added 1 mock(s) for T13289: Plans: 2019 Week 21-23 Bonus Content: M1479: Meow Meow.
Jun 4 2019, 7:38 PM · Plans
epriestley added 1 task(s) for M1479: Meow Meow: T13289: Plans: 2019 Week 21-23 Bonus Content.
Jun 4 2019, 7:38 PM
epriestley added a comment to T13289: Plans: 2019 Week 21-23 Bonus Content.

  • Jun 4 2019, 7:37 PM · Plans
    epriestley updated images of M1479: Meow Meow.
    Jun 4 2019, 7:36 PM
    epriestley added a revision to T13160: Support <colgroup> in the verbose remarkup <table> syntax for specifying column widths: Unknown Object (Differential Revision).
    Jun 4 2019, 4:32 PM · Remarkup
    epriestley closed T13300: When a "Draft" revision is closed into "Published" (not "Abandoned"), it should immediately promote as Resolved by committing Restricted Diffusion Commit.
    Jun 4 2019, 1:56 PM · Diffusion, Differential

    May 31 2019

    epriestley added a revision to T13298: Improve Herald profiling information available in transcripts: Unknown Object (Differential Revision).
    May 31 2019, 2:59 PM · Herald
    epriestley added a revision to T13298: Improve Herald profiling information available in transcripts: Unknown Object (Differential Revision).
    May 31 2019, 2:33 PM · Herald
    epriestley added a revision to T13300: When a "Draft" revision is closed into "Published" (not "Abandoned"), it should immediately promote: Unknown Object (Differential Revision).
    May 31 2019, 1:18 PM · Diffusion, Differential

    May 30 2019

    epriestley updated the task description for T13289: Plans: 2019 Week 21-23 Bonus Content.
    May 30 2019, 11:48 PM · Plans
    epriestley added a comment to T13297: "vi" and "vim" may exit with errors even though edits succeed.

    I'm just going to close this in the hope it fades quietly into the night now that it has been "fixed".

    May 30 2019, 11:47 PM · Arcanist
    epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

    D20563 probably fixes mail. I don't think (?) it will fix notifications but haven't hunted that down yet.

    May 30 2019, 11:15 PM · Transactions
    epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Unknown Object (Differential Revision).
    May 30 2019, 10:37 PM · Transactions
    epriestley updated the task description for T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.
    May 30 2019, 7:10 PM · Quicksand
    epriestley renamed T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes from There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changs to There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.
    May 30 2019, 7:09 PM · Transactions
    epriestley created T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.
    May 30 2019, 7:09 PM · Transactions
    epriestley added a comment to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.

    This likely affects other similar dialogs. The fix is to duct-tape something in Quicksand to something in the "click an area to click a link" behavior, most likely.

    May 30 2019, 7:07 PM · Quicksand