Page MenuHomePhabricator
Feed Advanced Search

Feb 17 2017

epriestley added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

(I'm not sure how "Compatibility Mode" and the DOM inspector are interacting, so maybe I'm just using it incorrectly.)

Feb 17 2017, 5:59 PM · Internet Explorer, Design
epriestley added a comment to T12289: When a user is attending multiple overlapping events, computation of availability state may not be accurate.

Here's the note in PhabricatorPeopleQuery about this:

Feb 17 2017, 5:42 PM · Calendar, Bug Report
epriestley added a comment to T12288: Phabricator OAuth 2.0 initiating SSO login from a Third Party website.

I don't think this is worth bringing upstream. It very occasionally saves users one click on an otherwise straightforward workflow, but requires significant complexity and maintenance from the upstream.

Feb 17 2017, 5:40 PM · Auth, Feature Request
epriestley added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

According to compatibility mode on my machine (Windows 7, IE 11, in IE10 mode), there is no crumb bug in IE10 if float: left is removed. This isn't consistent with T5902 or with other results I got (e.g., this thing says there is such a bug).

Feb 17 2017, 5:29 PM · Internet Explorer, Design
epriestley added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

Are you OK with this plan?

Feb 17 2017, 5:20 PM · Internet Explorer, Design
epriestley added a comment to T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).

I made some effort to fix this, but without much headway:

Feb 17 2017, 5:16 PM · Internet Explorer, Design
epriestley added a comment to T12157: Make it visibly clearer that a Phabricator user account has been disabled.

D17374 makes hovercards a little bit more clear too, although that might get some design feedback still (see "Horse" here, for example):

Feb 17 2017, 4:08 PM · Profile, Feature Request
epriestley added a revision to T12157: Make it visibly clearer that a Phabricator user account has been disabled: Unknown Object (Differential Revision).
Feb 17 2017, 3:25 PM · Profile, Feature Request
epriestley added a revision to T12268: Make the grey "Unverified User" dot accurately reflect the new universal behavior: Unknown Object (Differential Revision).
Feb 17 2017, 3:25 PM · Mail, People
epriestley added a comment to T11172: Unable to configure Arcanist as per the quick start tutorial.

In the future (T550) we plan to allow arc commands to use an SSH key. I'm not sure if that will reduce or increase confusion.

Feb 17 2017, 3:16 PM · Documentation, Bug Report
epriestley created T12287: Crumb width on profile pages is extremely conservative in Safari (always chops off the last couple letters?).
Feb 17 2017, 3:10 PM · Internet Explorer, Design
epriestley added a comment to T11172: Unable to configure Arcanist as per the quick start tutorial.

You should be able to can arc install-certificate --conduit-uri=https://whatever to install a certificate for an arbitrary install with no other configuration.

Feb 17 2017, 3:05 PM · Documentation, Bug Report
epriestley merged task T12286: Passphrase Conduit API does not specify secret for Note credential type into T5553: Allow SSL certificate to be stored in Passphrase.
Feb 17 2017, 2:35 PM · Bug Report
epriestley merged T12286: Passphrase Conduit API does not specify secret for Note credential type into T5553: Allow SSL certificate to be stored in Passphrase.
Feb 17 2017, 2:35 PM · Passphrase
epriestley added a comment to T12286: Passphrase Conduit API does not specify secret for Note credential type.

What is your use case for API access to notes?

Feb 17 2017, 2:30 PM · Bug Report
epriestley closed T12284: Wrong notification label (subtask status update) as Resolved by committing Restricted Diffusion Commit.
Feb 17 2017, 2:09 PM · Bug Report
epriestley added a comment to T12284: Wrong notification label (subtask status update).

D17373 should fix the typo.

Feb 17 2017, 1:34 PM · Bug Report
epriestley added a revision to T12284: Wrong notification label (subtask status update): Unknown Object (Differential Revision).
Feb 17 2017, 1:26 PM · Bug Report

Feb 16 2017

epriestley closed T12281: Exception thrown when adding a form to the Favorites menu as Resolved by committing Restricted Diffusion Commit.
Feb 16 2017, 11:45 PM · Bug Report
epriestley added a comment to T12281: Exception thrown when adding a form to the Favorites menu.

Thanks! Your reproduction instructions were extremely clear and made hunting this down straightforward. D17372 should resolve this.

Feb 16 2017, 11:19 PM · Bug Report
epriestley added a revision to T12281: Exception thrown when adding a form to the Favorites menu: Unknown Object (Differential Revision).
Feb 16 2017, 11:17 PM · Bug Report
epriestley added a comment to T12282: "Phabricator Media" wiki page has old logos.

I gave the goat some leaves to eat because maybe goats eat leaves?

Feb 16 2017, 11:00 PM · Design
epriestley updated the task description for T12282: "Phabricator Media" wiki page has old logos.
Feb 16 2017, 10:59 PM · Design
epriestley created T12282: "Phabricator Media" wiki page has old logos.
Feb 16 2017, 10:58 PM · Design
epriestley added a comment to T9525: Provide a reasonable general use logo.

Those are old now I guess, I'll file something to see if @chad can fix it because I am not licensed to pixel.

Feb 16 2017, 10:57 PM · Public & Media Relations
epriestley added a comment to T12279: Timezone Asia/Jerusalem incorrectly listed under UTC-0.

Well by the time you divide false into 3600 pieces you can't have much left!

Feb 16 2017, 10:55 PM · Bug Report
zhiyuchen awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Feb 16 2017, 7:44 PM · Workboards (v3), Restricted Project
mbenua-mparticle awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Love token.
Feb 16 2017, 7:43 PM · Workboards (v3), Restricted Project
epriestley added a comment to T12263: Javascript licence.

See LICENSE for license information:

Feb 16 2017, 7:26 PM · Feature Request
epriestley added a revision to T2794: Implement Phage (like Hypershell): Unknown Object (Differential Revision).
Feb 16 2017, 1:50 PM · Phage, Infrastructure
epriestley added a comment to T11989: Wage an endless war against HP Printer Fax Support.

Recently, we've started seeing a handful of attacks where human users appear to read a discussion, formulate a human-sounding, contextual reply (e.g., discussing the thread topic in what appears to be a human way, just with a very shallow understanding of the issue), and then include a link to a site offering various services (mostly essay writing?) in the footer.

Feb 16 2017, 12:29 PM · Abuse
epriestley removed a watcher for Feature Request: • ilmked0514.
Feb 16 2017, 12:09 PM
epriestley removed a watcher for Feature Request: ps625353.
Feb 16 2017, 12:09 PM
epriestley removed a watcher for Feature Request: • tawels74.
Feb 16 2017, 12:09 PM
CodingNickNick awarded T2393: Adds a "Needs Verification" state to Audits a Love token.
Feb 16 2017, 11:40 AM · Restricted Project, Audit
epriestley updated subscribers of T12276: Allow to configure arc commit message.

Also, please work with other KDE project members to discuss KDE priorities and needs downstream, then have a single point of contact (nominally, @bcooksley) bring them upstream once there is consensus on the nature and priority of problems. Currently, you, @abika, and @bcooksley are all independently interacting with this upstream.

Feb 16 2017, 12:24 AM · KDE, Feature Request
avivey awarded T12271: When users accept a revision, allow them to accept for a subset of package/project reviewers they could accept for a Yellow Medal token.
Feb 16 2017, 12:24 AM · Audit, Restricted Project, Differential

Feb 15 2017

epriestley added a comment to T12276: Allow to configure arc commit message.

What specific modifications are you interested in making to the messages?

Feb 15 2017, 11:55 PM · KDE, Feature Request
epriestley merged T12274: Find/filter inline comments that are not "Done" into T8909: Provide ability to hide all inline or done comments.
Feb 15 2017, 10:35 PM · Inline Comments, Prioritized, Restricted Project, Restricted Project, Differential, FreeBSD
epriestley merged task T12274: Find/filter inline comments that are not "Done" into T8909: Provide ability to hide all inline or done comments.
Feb 15 2017, 10:35 PM · Differential, Feature Request
epriestley added a comment to T2794: Implement Phage (like Hypershell).

I'm making some amount of progress here, although I've only made it slightly past "run SSH in a loop" so far.

Feb 15 2017, 9:42 PM · Phage, Infrastructure
epriestley edited the content of Object Name Prefixes / Monograms.
Feb 15 2017, 9:35 PM
epriestley added a comment to T11050: Allow resigning from a review you are not directly responsible for.

Worth mentioning is that you can query for exact(username) to get a dashboard for only individual reviews, but this is a partial solution; there's no way to get a dashboard for "all my stuff, except stuff I've explicitly said I don't care about".

Feb 15 2017, 9:31 PM · Prioritized, Restricted Project, Differential, Feature Request
epriestley added a comment to T11050: Allow resigning from a review you are not directly responsible for.

This task has meandered a bit, but the current behavior is this:

Feb 15 2017, 9:30 PM · Prioritized, Restricted Project, Differential, Feature Request
epriestley added a project to T12272: Allow users to accept on behalf of packages they control a containing package for: Owners.
Feb 15 2017, 9:02 PM · Owners, Audit, Restricted Project, Differential
epriestley created T12272: Allow users to accept on behalf of packages they control a containing package for.
Feb 15 2017, 9:02 PM · Owners, Audit, Restricted Project, Differential
epriestley added a comment to T12270: Unbeta Badges.

I'm fine with just declining to implement that, it just seems like something we should fix if we're calling it a bug.

Feb 15 2017, 8:36 PM · Badges (v0)
epriestley created T12271: When users accept a revision, allow them to accept for a subset of package/project reviewers they could accept for.
Feb 15 2017, 8:32 PM · Audit, Restricted Project, Differential
epriestley created T12268: Make the grey "Unverified User" dot accurately reflect the new universal behavior.
Feb 15 2017, 11:19 AM · Mail, People
epriestley added a comment to T12266: arc set-config encoding XXX not work.

I think it was historically possible for third-party code to read custom config, but not possible for them to add new settings until D16928 (Nov 2016), which might be why we don't warn about bad values. Should be fine to do so now, though.

Feb 15 2017, 11:13 AM · Bug Report
epriestley edited this Badge.
Feb 15 2017, 11:07 AM
epriestley closed T11114: Move Differential to EditEngine as Resolved.

I think we've ended up in a fairly stable place here, and remaining work generally has existing tasks motivating it in terms of features (like T10967 for reviewers, T10574 for acceptance rules, T2543 for revision states, various tasks for inlines, T9713 for commit message parsing). Obsoleted transaction types also need migrations (but they should happen alongside the migrations that other features require so we're minimizing the number of weeks with migrations that need special planning) and this dumb mess got left in PhabricatorModularTransaction.php and will probably be there for an uncomfortably long time:

Feb 15 2017, 1:23 AM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential

Feb 14 2017

epriestley added a comment to T12240: Best of Both Worlds: one email to everyone with privacy controls and good threading.

We'll probably implement a Phabricator-side mute-this-thread in connection with T10448, I'm just guessing that some of the users complaining that GMail mute doesn't work won't consider clicking a link in the email body, then clicking "Mute Thread" to be an acceptable solution.

Feb 14 2017, 7:01 PM · Restricted Project, Mail, Feature Request
epriestley added a comment to T12240: Best of Both Worlds: one email to everyone with privacy controls and good threading.

I'm pretty sure we can't generate our own Message-ID with Amazon SES, at least -- although we use the HTTP API, not the SMTP API, and maybe the rules differ. Not sure about other providers.

Feb 14 2017, 6:56 PM · Restricted Project, Mail, Feature Request
epriestley closed T12263: Javascript licence as Invalid.

See the LICENSE file for license information.

Feb 14 2017, 6:49 PM · Feature Request
epriestley edited the content of Starmap.
Feb 14 2017, 3:25 PM
epriestley added a revision to T2794: Implement Phage (like Hypershell): Unknown Object (Differential Revision).
Feb 14 2017, 3:17 PM · Phage, Infrastructure
epriestley added a comment to T12178: Users can send messages to Conpherence rooms they do not have CAN_JOIN permission for.

(Or we should just remove CAN_JOIN -- is there any actual use case for read-only rooms? If there is, shouldn't users still be able to join those rooms in order to follow them, just not send messages?)

Feb 14 2017, 3:08 PM · Conpherence (v4), Security
epriestley added a comment to T12178: Users can send messages to Conpherence rooms they do not have CAN_JOIN permission for.

A related issue is that CAN_EDIT does not imply CAN_JOIN, so users with CAN_EDIT but no CAN_JOIN can get a policy error while trying to join.

Feb 14 2017, 3:07 PM · Conpherence (v4), Security
epriestley added a comment to T12258: Diviner atom document status inverted when searching.

(Your diagnosis of the root cause was also correct.)

Feb 14 2017, 12:05 PM · Diviner, Bug Report
epriestley added a comment to T12258: Diviner atom document status inverted when searching.

I deployed D17346 and rebuilt the indexes on this install with bin/search index --type DivinerLiveSymbol and behavior now seems correct. Thanks for the report!

Feb 14 2017, 12:05 PM · Diviner, Bug Report
epriestley added a comment to T12013: Enabling zlib.output_compression results in wrong Content-Length.

Please file a new task following the instructions in Contributing Bug Reports if you'd like us to take a look. We need all that information before we can start looking at a bug.

Feb 14 2017, 12:00 PM · Bug Report
epriestley added a subtask for T12101: Phabricator PHP 7 Compatibility: T12196: PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts.
Feb 14 2017, 11:54 AM · Infrastructure, Guides
epriestley added a parent task for T12196: PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts: T12101: Phabricator PHP 7 Compatibility.
Feb 14 2017, 11:54 AM · Bug Report
epriestley renamed T12196: PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts from `arc liberate` fails on Phabricator miraculously with PHP 7.1 to PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts.
Feb 14 2017, 11:54 AM · Bug Report
epriestley merged task T12262: ./bin/repository update - throws segfault into T12196: PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts.
Feb 14 2017, 11:53 AM · Bug Report
epriestley merged T12262: ./bin/repository update - throws segfault into T12196: PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts.
Feb 14 2017, 11:53 AM · Bug Report
epriestley added a comment to T12262: ./bin/repository update - throws segfault.

I'm going to merge this into T12196; this is almost certainly an issue with PHP7. Upgrading to PHP 7.1.2 may resolve it.

Feb 14 2017, 11:53 AM · Bug Report

Feb 13 2017

epriestley closed T12237: Amazon SES has suspended outbound mail from secure.phabricator.com as Resolved.

Possible followups from D17344:

Feb 13 2017, 11:06 PM · Mail, Phacility, Ops
epriestley set the image for KDE to F2868791: profile.
Feb 13 2017, 11:03 PM
epriestley closed T12258: Diviner atom document status inverted when searching as Resolved by committing Restricted Diffusion Commit.
Feb 13 2017, 9:12 PM · Diviner, Bug Report
epriestley added a project to T12258: Diviner atom document status inverted when searching: Diviner.
Feb 13 2017, 9:08 PM · Diviner, Bug Report
epriestley added a comment to T10538: Bridge GitHub to Phabricator.

See Planning for help with timelines and status.

Feb 13 2017, 9:07 PM · Nuance, GitHub
epriestley added a revision to T12258: Diviner atom document status inverted when searching: Unknown Object (Differential Revision).
Feb 13 2017, 9:07 PM · Diviner, Bug Report
epriestley closed T12255: Landing differential from other author does not preserve ownership as Invalid.

See downstream for general discussion. See T12256 / T12257 for upstream followups.

Feb 13 2017, 8:45 PM · KDE, Bug Report
epriestley created T12257: Add an "author email" field to the "Differential Diff" Herald rule.
Feb 13 2017, 8:42 PM · KDE, Herald, Differential
epriestley created T12256: Parse multiple commits and commit metadata from "git show" and "git format-patch".
Feb 13 2017, 8:29 PM · Differential, KDE
epriestley closed T10105: Disabled account should disable all outbound mail as Wontfix.

Example: I disabled a bogus user when the "confirm your e-mail" MTA job was on its ~200th attempt. It's now at its ~240th attempt :)

Feb 13 2017, 6:42 PM · llvm, Restricted Project, Mail, Bug Report
epriestley added a project to T12255: Landing differential from other author does not preserve ownership: KDE.
Feb 13 2017, 6:08 PM · KDE, Bug Report
epriestley added a revision to T12237: Amazon SES has suspended outbound mail from secure.phabricator.com: Unknown Object (Differential Revision).
Feb 13 2017, 5:58 PM · Mail, Phacility, Ops
epriestley updated subscribers of T12255: Landing differential from other author does not preserve ownership.

This is a copy of downstream https://phabricator.kde.org/T5242. See that task for discussion.

Feb 13 2017, 5:54 PM · KDE, Bug Report
epriestley renamed T11035: Only evaluate newly added mentions/subscribers when applying transactions to remarkup blocks from Only evaluate newly added mentions when applying transactions to remarkup blocks to Only evaluate newly added mentions/subscribers when applying transactions to remarkup blocks.
Feb 13 2017, 4:46 PM · Prioritized, Wikimedia, Remarkup
epriestley created P2031 sell_private_user_data_to_the_highest_bidder.diff.
Feb 13 2017, 4:24 PM
epriestley merged T12254: re-subscribes on edit into T11035: Only evaluate newly added mentions/subscribers when applying transactions to remarkup blocks.
Feb 13 2017, 3:57 PM · Prioritized, Wikimedia, Remarkup
epriestley merged task T12254: re-subscribes on edit into T11035: Only evaluate newly added mentions/subscribers when applying transactions to remarkup blocks.
Feb 13 2017, 3:57 PM · Bug Report
epriestley added a comment to T12253: null in remarkup rather than comment reference when quoting a comment in audit.

Thanks, this should now be fixed in HEAD of master. Let us know if you run into anything else.

Feb 13 2017, 3:44 PM · Diffusion, Bug Report
epriestley closed T12253: null in remarkup rather than comment reference when quoting a comment in audit as Resolved by committing Restricted Diffusion Commit.
Feb 13 2017, 3:44 PM · Diffusion, Bug Report
epriestley added a revision to T12253: null in remarkup rather than comment reference when quoting a comment in audit: Unknown Object (Differential Revision).
Feb 13 2017, 3:42 PM · Diffusion, Bug Report
epriestley claimed T12253: null in remarkup rather than comment reference when quoting a comment in audit.
Feb 13 2017, 3:38 PM · Diffusion, Bug Report
epriestley closed T12245: When creating an auth provider, the header says "Enabled" but you haven't created it yet as Resolved.

Resolved by D17337.

Feb 13 2017, 3:03 PM · Auth
epriestley closed T12252: Unhandled Exception ("RuntimeException") at Conduit Call Logs page as Resolved by committing Restricted Diffusion Commit.
Feb 13 2017, 2:24 PM · Conduit, Bug Report
epriestley claimed T12252: Unhandled Exception ("RuntimeException") at Conduit Call Logs page.
Feb 13 2017, 2:23 PM · Conduit, Bug Report
epriestley removed a watcher for Bug Report: • greedhector28.
Feb 13 2017, 2:22 PM
epriestley removed a watcher for Bug Report: • Alexmoon2.
Feb 13 2017, 2:22 PM
epriestley added a revision to T12252: Unhandled Exception ("RuntimeException") at Conduit Call Logs page: Unknown Object (Differential Revision).
Feb 13 2017, 2:18 PM · Conduit, Bug Report
epriestley added a comment to T12177: Provide time-to-review statistics and reporting in Facts.

The WMF downstream has some similar discussion here and in linked tasks:

Feb 13 2017, 1:17 PM · Customer Impact, Differential, Restricted Project, Facts, Feature Request
epriestley added a project to T12177: Provide time-to-review statistics and reporting in Facts: Differential.
Feb 13 2017, 1:06 PM · Customer Impact, Differential, Restricted Project, Facts, Feature Request
epriestley renamed T12177: Provide time-to-review statistics and reporting in Facts from Use case for Facts and ETL to Provide time-to-review statistics and reporting in Facts.
Feb 13 2017, 1:06 PM · Customer Impact, Differential, Restricted Project, Facts, Feature Request
epriestley triaged T12251: Add author information when creating a build in Buildkite as Wishlist priority.
Feb 13 2017, 12:56 PM · Harbormaster, Feature Request
epriestley edited the content of Changelog.
Feb 13 2017, 12:44 PM