Page MenuHomePhabricator
Feed All Stories

Feb 14 2017

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
faulconbridge added a comment to T12258: Diviner atom document status inverted when searching.

+1 for random grepping. Thanks for the fix!

Feb 14 2017, 1:00 PM · Diviner, Bug Report
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
ivo added a comment to T11816: Calendar: Feedback and Suggestions.

When accessing an ICS export of all events I can see (https://secure.phabricator.com/calendar/export/3/):

Unhandled Exception ("Exception")	
DateTime::__construct(): Failed to parse time string (@) at position 0 (@): Unexpected character
Feb 14 2017, 11:46 AM · Restricted Project, Calendar
BYK added a comment to T8298: Fix Windows Escaping.

@chad - How can I be involved with any progress on this front then since now it is planned?

Feb 14 2017, 8:29 AM · Windows
der.internethering created T12262: ./bin/repository update - throws segfault.
Feb 14 2017, 7:50 AM · Bug Report
chad added a comment to T8298: Fix Windows Escaping.

See T11429

Feb 14 2017, 7:20 AM · Windows
OCram added a comment to T8298: Fix Windows Escaping.

Do you think there are any intentions to fix this issue for windows users? At least in far future?
Me and my colleges are still working with an old arcanist version 737f5c0d. If this workaround breaks, there would be no way for windows users to work with arcanist. Except using a fork containing D15675.

Feb 14 2017, 7:08 AM · Windows
codeb2cc added a comment to T12013: Enabling zlib.output_compression results in wrong Content-Length.

I think this compression had messed up with HTTP served Git repository. Git complains with

Feb 14 2017, 6:53 AM · Bug Report
chad added a revision to T10145: Adding a panel to a dashboard multiple times: Unknown Object (Differential Revision).
Feb 14 2017, 4:34 AM · Dashboards (v2), Bug Report
chad claimed T10145: Adding a panel to a dashboard multiple times.
Feb 14 2017, 3:46 AM · Dashboards (v2), Bug Report
chad added a comment to T10145: Adding a panel to a dashboard multiple times.

I presume there is no logical need to add the same panel twice to a dashboard.

Feb 14 2017, 3:45 AM · Dashboards (v2), Bug Report
chad closed T10612: Editing a panel causes it to duplicate in dashboard as Resolved by committing Restricted Diffusion Commit.
Feb 14 2017, 2:52 AM · Dashboards (v2), Bug Report
chad closed T10612: Editing a panel causes it to duplicate in dashboard, a subtask of T12207: Sort out the great Dashboard column editing mess, as Resolved.
Feb 14 2017, 2:52 AM · Dashboards (v2)
chad added a revision to T10612: Editing a panel causes it to duplicate in dashboard: Unknown Object (Differential Revision).
Feb 14 2017, 2:34 AM · Dashboards (v2), Bug Report
alexmv closed T10881: git diff parsing fails with --submodule=log as Resolved.
Feb 14 2017, 1:41 AM · Contributor Onboarding, Arcanist, Bug Report
faulconbridge removed a project from T12261: helll my test: Badges.
Feb 14 2017, 12:55 AM
faulconbridge closed T12261: helll my test as Spite.
Feb 14 2017, 12:54 AM
• liuyi created T12261: helll my test.
Feb 14 2017, 12:53 AM
chad added a revision to T10390: Dashboards v2 (UX): Unknown Object (Differential Revision).
Feb 14 2017, 12:33 AM · Dashboards (v2)
chad claimed T10390: Dashboards v2 (UX).
Feb 14 2017, 12:30 AM · Dashboards (v2)
chad triaged T10390: Dashboards v2 (UX) as High priority.
Feb 14 2017, 12:29 AM · Dashboards (v2)

Feb 13 2017

chad closed T12128: Favorites Menu Errata as Resolved.

Seems functional.

Feb 13 2017, 11:55 PM · Profile Menu Item
chad merged T12260: Report on historical assignment of points into T1562: Build "Facts", an ETL pipeline and charting application.
Feb 13 2017, 11:47 PM · Restricted Project, Wikimedia, Facts
chad merged task T12260: Report on historical assignment of points into T1562: Build "Facts", an ETL pipeline and charting application.
Feb 13 2017, 11:46 PM · Feature Request
chad merged T12259: Report on delta in task list between two points in time into T1562: Build "Facts", an ETL pipeline and charting application.
Feb 13 2017, 11:46 PM · Restricted Project, Wikimedia, Facts
chad merged task T12259: Report on delta in task list between two points in time into T1562: Build "Facts", an ETL pipeline and charting application.
Feb 13 2017, 11:46 PM · Maniphest, Feature Request
jaufrecht created T12260: Report on historical assignment of points.
Feb 13 2017, 11:25 PM · Feature Request
jaufrecht created T12259: Report on delta in task list between two points in time.
Feb 13 2017, 11:19 PM · Maniphest, Feature Request
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
jaufrecht added a comment to T10339: Figure out the UI for Maniphest task search by points.

Here is an example of a report that is satisfying several use cases from the comment above, https://secure.phabricator.com/T10339#159098. Those use cases, fleshed out a bit, include:

Feb 13 2017, 11:05 PM · Maniphest
epriestley set the image for KDE to F2868791: profile.
Feb 13 2017, 11:03 PM
jaufrecht added a comment to T4434: Track object closed/completed date and actor.

More use cases:

Feb 13 2017, 10:56 PM · Restricted Project, Maniphest, Projects
chad closed T12160: Lightbox asks user to log in if logged out as Resolved by committing Restricted Diffusion Commit.
Feb 13 2017, 9:54 PM · Lightbox, Bug Report
chad added a comment to T8418: 'door with arrow' login / logout icon needs an explanatory mouseover.

This now just says "Login" in text.

Feb 13 2017, 9:53 PM
chad added a revision to T12160: Lightbox asks user to log in if logged out: Unknown Object (Differential Revision).
Feb 13 2017, 9:49 PM · Lightbox, Bug Report
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
chad added a comment to T10538: Bridge GitHub to Phabricator.

Hard to say since it fell off the roadmap a year ago, but we do have rough plans to ship Nuance (pre-requisite) in the shorter term.

Feb 13 2017, 9:10 PM · Nuance, GitHub
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
mbenua-mparticle added a comment to T10538: Bridge GitHub to Phabricator.

Where does this feature stand? I wasn't able to find any documentation on how to use this, but it also looks that a non-trivial amount of work has been completed towards this goal. Is this something still in the immediate roadmap? Being able to easily integrate GitHub issues into Phabricator (and back) is a big roadblock to adoption for us.

Feb 13 2017, 9:06 PM · Nuance, GitHub
faulconbridge created T12258: Diviner atom document status inverted when searching.
Feb 13 2017, 8:49 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
chad changed the image for Image Macro "shipitfire".
Feb 13 2017, 7:12 PM
chad claimed T12160: Lightbox asks user to log in if logged out.
Feb 13 2017, 6:42 PM · Lightbox, Bug Report
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
chad edited the content of 2017 Week 6 (Mid February).
Feb 13 2017, 6:32 PM
abika added a comment to T12255: Landing differential from other author does not preserve ownership.

Ah, thanks! Good to know what my own community is doing.

Feb 13 2017, 6:25 PM · KDE, 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
abika created T12255: Landing differential from other author does not preserve ownership.
Feb 13 2017, 5:41 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
ofbeaton added a comment to T11035: Only evaluate newly added mentions/subscribers when applying transactions to remarkup blocks.

Can we please add the keyword subscribe or subscribers in this task title? I looked through 2 pages of subscribers search before filing a dup. I know now I was searching the wrong term, mention would have given it to me instead. But they are related.

Feb 13 2017, 4:11 PM · Prioritized, Wikimedia, Remarkup
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
ofbeaton added a comment to T4411: Adding a CC to a Maniphest Task should give View rights for that user.

We would also like this behaviour, and currently use a workaround to accomplish it (which is why we never came here looking for a fix). We created a custom Create Task form that sets the default view policy to include subscribers.

Feb 13 2017, 3:50 PM · Wikimedia, Policy
ofbeaton created T12254: re-subscribes on edit.
Feb 13 2017, 3:45 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
mpickering created T12253: null in remarkup rather than comment reference when quoting a comment in audit.
Feb 13 2017, 3:35 PM · Diffusion, Bug Report
sergio.castro added a comment to T12222: Add custom fields to the "Task Action..." menu.

Hi Evan,

Feb 13 2017, 3:32 PM · Maniphest, Feature Request
chad closed T4984: Improve design of dashboard browse list? as Resolved by committing Restricted Diffusion Commit.
Feb 13 2017, 3:24 PM · Dashboards (v2), Design
chad closed T4984: Improve design of dashboard browse list?, a subtask of T4980: Move Dashboards to beta, as Resolved.
Feb 13 2017, 3:24 PM · Dashboards
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
ivo created T12252: Unhandled Exception ("RuntimeException") at Conduit Call Logs page.
Feb 13 2017, 1:13 PM · Conduit, Bug Report
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
ivo created T12251: Add author information when creating a build in Buildkite.
Feb 13 2017, 12:50 PM · Harbormaster, Feature Request
epriestley edited the content of Changelog.
Feb 13 2017, 12:44 PM
epriestley created 2017 Week 6 (Mid February).
Feb 13 2017, 12:43 PM
ivo added a project to T12250: Add repository short name as a variable in build steps: Harbormaster.
Feb 13 2017, 12:31 PM · Harbormaster, Feature Request
ivo created T12250: Add repository short name as a variable in build steps.
Feb 13 2017, 12:31 PM · Harbormaster, Feature Request
epriestley closed T12249: Test-BR as Invalid.
Feb 13 2017, 10:32 AM · Bug Report
remusvrm updated Q562: How can I change the default Space? from to How can I change the default Space?.
Feb 13 2017, 10:09 AM · Spaces
• bilalramazanoglu created T12249: Test-BR.
Feb 13 2017, 9:36 AM · Bug Report
remusvrm closed Q529: Why I can not tag anymore a milestone? as resolved.
Feb 13 2017, 9:28 AM