Page MenuHomePhabricator
Feed Advanced Search

Jul 28 2017

epriestley added a comment to T12942: Add a "Binary Versions" page to Config.

See also PHI22.

Jul 28 2017, 6:18 PM · Config
epriestley created T12942: Add a "Binary Versions" page to Config.
Jul 28 2017, 6:15 PM · Config
epriestley added a comment to T2383: Run PHP environment checks from CLI.

T12941 is a third case of a user with no mysql or mysqli.

Jul 28 2017, 10:55 AM · Setup
epriestley merged T12941: missing mysql extension incredibly difficult to debug into T2383: Run PHP environment checks from CLI.
Jul 28 2017, 10:54 AM · Setup
epriestley merged task T12941: missing mysql extension incredibly difficult to debug into T2383: Run PHP environment checks from CLI.
Jul 28 2017, 10:54 AM

Jul 27 2017

epriestley added a comment to T12940: Account Setup Issues shown to users before signing TOS.

The particulars in T8918 are slightly different, but the root issue (showing too much stuff to users with partial sessions) is the same.

Jul 27 2017, 9:24 PM
epriestley merged T12940: Account Setup Issues shown to users before signing TOS into T8918: Header shows number of notifications and various other controls on the 2FA auth screen.
Jul 27 2017, 9:23 PM · Security, Auth
epriestley merged task T12940: Account Setup Issues shown to users before signing TOS into T8918: Header shows number of notifications and various other controls on the 2FA auth screen.
Jul 27 2017, 9:23 PM
epriestley merged T11639: InvalidArgumentException when clicking "Show Older Changes" on diff into T11843: Hitting "show older changes" throws an exception on DifferentialTransactionView::setLeftDiff() with Quicksand enabled.
Jul 27 2017, 4:53 PM · Quicksand, Bug Report
epriestley merged task T11639: InvalidArgumentException when clicking "Show Older Changes" on diff into T11843: Hitting "show older changes" throws an exception on DifferentialTransactionView::setLeftDiff() with Quicksand enabled.
Jul 27 2017, 4:53 PM · Restricted Project, Bug Report
epriestley closed T11835: Diffs/Patches should optionally not be generated for generated files until the item is expanded. as Invalid.

This is vague and may not actually identify a problem; I don't plan to pursue it.

Jul 27 2017, 4:50 PM · Diffusion, Feature Request
epriestley closed T11989: Wage an endless war against HP Printer Fax Support as Resolved.

We've closed registration on this install, mooting this.

Jul 27 2017, 4:45 PM · Abuse
epriestley closed T11989: Wage an endless war against HP Printer Fax Support, a subtask of T10215: Provide tools to combat and recover from abuse, as Resolved.
Jul 27 2017, 4:45 PM · Abuse
epriestley closed T12136: What to do with "Application Counts" with MenuItem, a subtask of T11957: Add MenuEngine to Home, as Wontfix.
Jul 27 2017, 4:42 PM · Profile Menu Item, Dashboards
epriestley closed T12136: What to do with "Application Counts" with MenuItem as Wontfix.

Dropping these doesn't seem to have caused any major pushback.

Jul 27 2017, 4:42 PM · Profile Menu Item
epriestley closed T12208: Collapsable Menu Item as Resolved.

This presumably counts as "not much interest" or should move to Discourse now or something, maybe.

Jul 27 2017, 4:32 PM · Profile Menu Item
epriestley merged T12278: Choose specific form on Create Subtask into T12588: create subtask side menu to allow creation of subtasks of different types to the parent task.
Jul 27 2017, 4:29 PM · Feature Request
epriestley merged task T12278: Choose specific form on Create Subtask into T12588: create subtask side menu to allow creation of subtasks of different types to the parent task.
Jul 27 2017, 4:29 PM · Maniphest, Feature Request
epriestley closed T12291: Diffusion doesn't display file moves/renames in an easily consumable/reviewable way as Wontfix.

This seems to mostly be a matter of taste, and we already have behavior similar to the behavior of git, which seems reasonable.

Jul 27 2017, 4:28 PM · Diffusion, Feature Request
epriestley closed T12292: Diffusion commit details references HEAD -> master broken as Invalid.

We don't have reproduction steps for this and thus can't move forward.

Jul 27 2017, 4:26 PM · Bug Report (Needs Information)
epriestley merged T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user into T12871: Setting a restricted dashboard as the topmost item in the global menu can lock some users out.
Jul 27 2017, 4:25 PM · Policy, Dashboards, Bug Report
epriestley merged task T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user into T12871: Setting a restricted dashboard as the topmost item in the global menu can lock some users out.
Jul 27 2017, 4:25 PM · Dashboards, Profile Menu Item, Bug Report
epriestley added a comment to T12404: Implement a first-party SMTP client.

This doesn't affect us, but more fuel on the fire -- XSS in example code which ships with PHPMailer (we don't distribute this code):

Jul 27 2017, 2:55 PM · Infrastructure, Mail
epriestley closed T12515: Upgrading: File Integrity Hashing and SHA1 as Resolved.

We appear to have survived this.

Jul 27 2017, 2:47 PM · Files, Guides, Security
epriestley closed T12560: Unintuitive interaction between autofixes and `arc diff` as Invalid.

Per above, this is not a bug.

Jul 27 2017, 2:43 PM · Bug Report
epriestley closed T12569: Conpherence rooms with no title now show nothing in the UI as Wontfix.

No one but me cares.

Jul 27 2017, 2:42 PM · Conpherence
epriestley closed T12572: Spend a week making Conpherence less bad as Resolved.

We kinda did some of this, it can live on in subtasks.

Jul 27 2017, 2:42 PM · Conpherence (v4)
epriestley closed T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation as Invalid.

This appears to lack reproduction steps, so we can't move forward.

Jul 27 2017, 2:40 PM · Bug Report
epriestley closed T12601: Set default arguments of `ArcanistWorkflow` as Invalid.
Jul 27 2017, 2:40 PM · Feature Request
epriestley closed T12629: Start daemons that should be running but aren't as Invalid.

We aren't going to implement a bin/phd start-missing-daemon command.

Jul 27 2017, 2:32 PM · Daemons, Feature Request
epriestley renamed T12891: Phacility does not currently offer instances in the Sydney region from Slow cloning over Phacility SSH for large repository to Phacility does not currently offer instances in the Sydney region.
Jul 27 2017, 2:18 PM · Feature Request
epriestley triaged T12891: Phacility does not currently offer instances in the Sydney region as Wishlist priority.

What is the network speed of transferring a similar file (e.g., a 400MB file from /dev/urandom) via ssh cat <file>?

Jul 27 2017, 2:17 PM · Feature Request
epriestley updated the task description for T12904: Support Pact Errata.
Jul 27 2017, 2:02 PM · Phacility, Support
epriestley triaged T12930: Firefox renders the open dropdown states for <select /> inputs with a "color" style in a primitive way as Wishlist priority.
Jul 27 2017, 2:01 PM · Firefox, Bug Report
epriestley added a project to T12930: Firefox renders the open dropdown states for <select /> inputs with a "color" style in a primitive way: Firefox.
Jul 27 2017, 2:00 PM · Firefox, Bug Report
epriestley renamed T12930: Firefox renders the open dropdown states for <select /> inputs with a "color" style in a primitive way from Dropdown styles menu on Firefox/OSX has changed to Firefox renders the open dropdown states for <select /> inputs with a "color" style in a primitive way.
Jul 27 2017, 2:00 PM · Firefox, Bug Report
epriestley reassigned T12930: Firefox renders the open dropdown states for <select /> inputs with a "color" style in a primitive way from epriestley to chad.

Per above, the decision to color the default state is explicit so this is likely just a design tradeoff in the short term. I can test things locally if you want to keep fishing.

Jul 27 2017, 1:59 PM · Firefox, Bug Report
epriestley added a comment to T12930: Firefox renders the open dropdown states for <select /> inputs with a "color" style in a primitive way.

There's probably no reason for us to have color: ... on the <select />

Jul 27 2017, 1:57 PM · Firefox, Bug Report
epriestley claimed T12930: Firefox renders the open dropdown states for <select /> inputs with a "color" style in a primitive way.

Is this the current status here?

Jul 27 2017, 1:40 PM · Firefox, Bug Report
epriestley merged T12939: Ponder seems to ignore all email settings into T10448: Modularize mail tags.
Jul 27 2017, 1:33 PM · Prioritized, Restricted Project, Mail, User Preferences, Owners, Feature Request
epriestley merged task T12939: Ponder seems to ignore all email settings into T10448: Modularize mail tags.
Jul 27 2017, 1:33 PM · Bug Report, Ponder
epriestley added a comment to T12939: Ponder seems to ignore all email settings.

I'm just going to merge this into T10448.

Jul 27 2017, 1:33 PM · Bug Report, Ponder
epriestley merged T12909: Calendar ignores email preferences if a user is invited via a project into T10448: Modularize mail tags.
Jul 27 2017, 1:31 PM · Prioritized, Restricted Project, Mail, User Preferences, Owners, Feature Request
epriestley merged task T12909: Calendar ignores email preferences if a user is invited via a project into T10448: Modularize mail tags.
Jul 27 2017, 1:31 PM · Calendar, Bug Report
epriestley added a comment to T12909: Calendar ignores email preferences if a user is invited via a project.

This almost certainly has nothing to do with being invited via a project. I'm just going to merge this into T10448, which will moot it.

Jul 27 2017, 1:30 PM · Calendar, Bug Report
epriestley added a comment to T12939: Ponder seems to ignore all email settings.

I suspect this is actually more like: actions on Ponder answers are not covered by the "Ponder Questions" email preferences, because a "Ponder Answer" is not the same type of object as a "Ponder Question".

Jul 27 2017, 1:26 PM · Bug Report, Ponder
yelirekim awarded T12935: Automatically detect merge-ability in Differential a Like token.
Jul 27 2017, 1:16 PM · Customer Impact, Harbormaster, Drydock, Differential
epriestley removed a member for Ops: • bnealey.
Jul 27 2017, 10:52 AM
epriestley removed a member for Ops: • Hektve.
Jul 27 2017, 10:52 AM
epriestley removed a member for Ops: • z4a4n2008lv.
Jul 27 2017, 10:52 AM
epriestley removed a watcher for Ops: • bnealey.
Jul 27 2017, 10:52 AM
epriestley removed a watcher for Ops: • Hektve.
Jul 27 2017, 10:52 AM

Jul 26 2017

• tomekj2ee awarded T12935: Automatically detect merge-ability in Differential a Like token.
Jul 26 2017, 7:56 PM · Customer Impact, Harbormaster, Drydock, Differential
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 26 2017, 5:50 PM · Phacility, Support
epriestley added a comment to T5505: Plan Legalpad v2.

I think it should be possible to backfill this from the transaction record, but definitely a bit involved. Holding it until we do history (and major/minor versions) makes sense.

Jul 26 2017, 5:04 PM · Restricted Project, Restricted Project

Jul 25 2017

epriestley added a comment to T12929: Tasks created via email don't get "created" feed stories. They say "renamed".

There's also bin/mail receive-test but that only accepts mail --to an existing object, not a random email address, right now. We could make that more flexible to make testing a little easier (raw_mail.txt must be a full piece of mail with proper headers and encoding, but bin/mail receive-test accepts just the plain text of a body and simulates all the headers/encoding/envelope stuff).

Jul 25 2017, 8:31 PM · Mail, Bug Report, Maniphest
epriestley added a comment to T12929: Tasks created via email don't get "created" feed stories. They say "renamed".

Something like:

Jul 25 2017, 8:29 PM · Mail, Bug Report, Maniphest
epriestley closed T12934: "Support" can produce a bogus "0 to 0" mana cost transaction as Resolved by committing Restricted Diffusion Commit.
Jul 25 2017, 3:19 PM · Support
epriestley added a revision to T12934: "Support" can produce a bogus "0 to 0" mana cost transaction: Unknown Object (Differential Revision).
Jul 25 2017, 2:09 PM · Support
epriestley updated the task description for T12935: Automatically detect merge-ability in Differential.
Jul 25 2017, 2:03 PM · Customer Impact, Harbormaster, Drydock, Differential
epriestley created T12935: Automatically detect merge-ability in Differential.
Jul 25 2017, 2:02 PM · Customer Impact, Harbormaster, Drydock, Differential
epriestley created T12934: "Support" can produce a bogus "0 to 0" mana cost transaction.
Jul 25 2017, 1:01 PM · Support
joshuaspence added a comment to T6524: Support "request is from IP / subnet" policy rule.

@hach-que, did you end up implementing a solution for this?

Jul 25 2017, 12:58 PM · Policy
epriestley closed T12893: Instance reports consistent bad repository device versions for clustered, observed Git repositories as Resolved by committing Restricted Diffusion Commit.
Jul 25 2017, 12:12 PM · Restricted Project, Clusters, Diffusion, Customer Impact
epriestley added a comment to T12933: Legalpad "Updated" date is wrong.

I think you're innocent and that it never worked but I only looked at it for like 2 minutes.

Jul 25 2017, 12:51 AM · Restricted Project, Bug Report

Jul 24 2017

epriestley added a comment to T12933: Legalpad "Updated" date is wrong.

There may be a bug with the body versioning (maybe it got broken by accident when we swapped to ModularTransactions, or maybe it never worked), but if there is we should fix that bug. The creation date and modification date of a body should always be the same, like they are for a Phriction document body.

Jul 24 2017, 11:51 PM · Restricted Project, Bug Report
epriestley added a revision to T12733: (2017 Week 20) Inline Comments Errata / Feedback: Unknown Object (Differential Revision).
Jul 24 2017, 11:31 PM · Inline Comments, Installing & Upgrading, Differential
epriestley added a comment to T12733: (2017 Week 20) Inline Comments Errata / Feedback.

Maybe I'll put it on A, I guess ("Toggle All"), so it's sort of near q, which is similar (collapse/expand).

Jul 24 2017, 9:48 PM · Inline Comments, Installing & Upgrading, Differential
epriestley added a comment to T12733: (2017 Week 20) Inline Comments Errata / Feedback.

If it's a toggle, what do you expect us to do if you press it with some inlines hidden and others visible?

Jul 24 2017, 8:59 PM · Inline Comments, Installing & Upgrading, Differential
epriestley added a revision to T12893: Instance reports consistent bad repository device versions for clustered, observed Git repositories: Unknown Object (Differential Revision).
Jul 24 2017, 6:43 PM · Restricted Project, Clusters, Diffusion, Customer Impact
epriestley added a comment to T12893: Instance reports consistent bad repository device versions for clustered, observed Git repositories.

See PHI14 for priority.

Jul 24 2017, 4:26 PM · Restricted Project, Clusters, Diffusion, Customer Impact
epriestley added a comment to T3626: Detection of language for some file types isn't good.

And one more.

Jul 24 2017, 4:23 PM · Customer Impact, Restricted Project, Differential
epriestley added a comment to T12929: Tasks created via email don't get "created" feed stories. They say "renamed".

The easiest fix is probably to add a TYPE_CREATE transaction into ReplyHandler or MailReceiver alongside the other creation transactions.

Jul 24 2017, 4:06 PM · Mail, Bug Report, Maniphest
epriestley added a project to T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential: Customer Impact.

This is now formally prioritized (see PHI14).

Jul 24 2017, 4:03 PM · Customer Impact, Restricted Project, Restricted Project, Prioritized, Differential
epriestley added a comment to T12932: GC old Differential changesets into File storage.

Only db001 and db002 are anywhere close to having problems with this and they still have a large amount of headroom, and we can bin/storage optimize at least some amount of additional headroom into existence, so this isn't "we're about to fall off a cliff" urgent or anything, but would be nice to tackle sooner than later.

Jul 24 2017, 3:53 PM · Files, Differential, Phacility, Ops
epriestley created T12932: GC old Differential changesets into File storage.
Jul 24 2017, 3:47 PM · Files, Differential, Phacility, Ops
epriestley renamed T12929: Tasks created via email don't get "created" feed stories. They say "renamed" from Sample to Tasks created via email don't get "created" feed stories. They say "renamed".
Jul 24 2017, 3:36 PM · Mail, Bug Report, Maniphest
epriestley added a comment to T3626: Detection of language for some file types isn't good.

We've had two customers hit this (both for Kotlin, although a config workaround is available).

Jul 24 2017, 3:35 PM · Customer Impact, Restricted Project, Differential
epriestley added a project to T3626: Detection of language for some file types isn't good: Customer Impact.
Jul 24 2017, 3:30 PM · Customer Impact, Restricted Project, Differential
epriestley added a comment to T12929: Tasks created via email don't get "created" feed stories. They say "renamed".

Is this a test or a bug report? If it's a bug report, how can I reproduce it?

Jul 24 2017, 3:00 PM · Mail, Bug Report, Maniphest
jcox awarded T9041: Control access to administrator capabilities with custom policies a Like token.
Jul 24 2017, 11:34 AM · FreeBSD, Policy, People
joshuaspence added a revision to T12393: Instance reporting possible lock issue around `git fetch` of clustered working copies: Unknown Object (Differential Revision).
Jul 24 2017, 11:00 AM · Diffusion
joshuaspence added a comment to T12393: Instance reporting possible lock issue around `git fetch` of clustered working copies.

I am also seeing this error and am trying to work on a fix.

Jul 24 2017, 9:23 AM · Diffusion

Jul 21 2017

epriestley edited the content of Changelog.
Jul 21 2017, 10:52 PM
epriestley edited the content of 2017 Week 29 (Late July).
Jul 21 2017, 10:52 PM
epriestley added a comment to T12931: Fallback for incorrect default branch in Diffusion could be nicer.

That is, my expectation when you have set "Default Branch" to "mmmmmmmmmmmmm" and visit the repository page is just:

Jul 21 2017, 10:16 PM · Feature Request, Diffusion
epriestley added a comment to T12931: Fallback for incorrect default branch in Diffusion could be nicer.

This is what it did in March 2016: T3607#163684

Jul 21 2017, 10:11 PM · Feature Request, Diffusion
epriestley added a comment to T12931: Fallback for incorrect default branch in Diffusion could be nicer.

Where is the exception coming from?

Jul 21 2017, 9:37 PM · Feature Request, Diffusion
epriestley added a comment to T12931: Fallback for incorrect default branch in Diffusion could be nicer.

I believe the issue here is:

Jul 21 2017, 5:41 PM · Feature Request, Diffusion
epriestley renamed T12931: Fallback for incorrect default branch in Diffusion could be nicer from Diffusion doesn't render tabs if master isn't present in git to Diffusion doesn't render tabs if a repository has commits and a "Default Branch" set to a branch which does not exist.
Jul 21 2017, 5:38 PM · Feature Request, Diffusion
epriestley added a comment to T12847: A Pathway Towards Private Clusters.

I think the minimum size is 2 hosts, in separate AZs, in one region. e.g., everything1 in us-west-1a and everything2 in us-west-1c, so that assumption is sound.

Jul 21 2017, 4:34 PM · Plans, Ops, Phacility
epriestley added a revision to T12733: (2017 Week 20) Inline Comments Errata / Feedback: Unknown Object (Differential Revision).
Jul 21 2017, 3:10 PM · Inline Comments, Installing & Upgrading, Differential
epriestley added a comment to T12733: (2017 Week 20) Inline Comments Errata / Feedback.

Well, I'm half mistaken. What you're seeing is a bug.

Jul 21 2017, 3:07 PM · Inline Comments, Installing & Upgrading, Differential
epriestley added a comment to T12733: (2017 Week 20) Inline Comments Errata / Feedback.

(Oh, no, I'm mistaken.)

Jul 21 2017, 2:52 PM · Inline Comments, Installing & Upgrading, Differential
epriestley added a comment to T12733: (2017 Week 20) Inline Comments Errata / Feedback.

That's supposed to leave an "Undo" bar, still want us to fix the bug?

Jul 21 2017, 2:42 PM · Inline Comments, Installing & Upgrading, Differential
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 21 2017, 2:38 PM · Phacility, Support
epriestley edited the content of Planning.
Jul 21 2017, 1:17 PM
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 21 2017, 11:35 AM · Phacility, Support
quark.zju awarded Blog Post: Write, Review, Merge, Publish: Phabricator Review Workflow a Like token.
Jul 21 2017, 2:57 AM

Jul 20 2017

epriestley added a revision to T12904: Support Pact Errata: Restricted Differential Revision.
Jul 20 2017, 10:27 PM · Phacility, Support