Page MenuHomePhabricator
Feed Advanced Search

Jan 9 2017

epriestley created T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.
Jan 9 2017, 5:32 PM · Installing & Upgrading, Differential
joshma added a comment to T9186: CloudFront configuration is more involved than the documentation implies.

FWIW this affected us as well, we switched from cloudflare back to cloudfront and hit this snag. The default settings do not forward Host header, we had to add that ourselves. Didn't need to change anything else, though - it seems that cloudfront performs loose SSL validation and is OK with the certificate/host mismatch. (Maybe it at least requires the host to match the origin domain - not sure.)

Jan 9 2017, 5:11 PM · Setup
epriestley closed T12074: conduit API extensions for subprojects and milestones. as Resolved.

I've also made these additional changes, not directly related to the original request:

Jan 9 2017, 4:54 PM · Feature Request
epriestley closed T12082: Differential emails are rendering literal HTML tags as Resolved by committing Restricted Diffusion Commit.
Jan 9 2017, 4:42 PM · Differential, Bug Report
epriestley added a comment to T12084: `differential.updaterevision` conduit method no longer updates custom fields.

I think this is mostly guidance I neglected to issue, sorry about that.

Jan 9 2017, 4:40 PM · Bug Report
jcox created T12084: `differential.updaterevision` conduit method no longer updates custom fields.
Jan 9 2017, 4:22 PM · Bug Report
epriestley added a revision to T11114: Move Differential to EditEngine: Unknown Object (Differential Revision).
Jan 9 2017, 4:21 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a comment to T12082: Differential emails are rendering literal HTML tags.

Thanks! D17162 should clean this up.

Jan 9 2017, 4:21 PM · Differential, Bug Report
epriestley added a revision to T11114: Move Differential to EditEngine: Unknown Object (Differential Revision).
Jan 9 2017, 4:20 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T12082: Differential emails are rendering literal HTML tags: Unknown Object (Differential Revision).
Jan 9 2017, 4:19 PM · Differential, Bug Report
epriestley added a revision to T12074: conduit API extensions for subprojects and milestones.: Unknown Object (Differential Revision).
Jan 9 2017, 3:40 PM · Feature Request
epriestley added a comment to T12074: conduit API extensions for subprojects and milestones..

These changes are now available in HEAD of master, and should be available on stable after Jan 13. master is also safe to upgrade into right now, although some T11114 stuff may churn later this week.

Jan 9 2017, 3:35 PM · Feature Request
epriestley merged T10382: Transaction abort when pushing via SSH to a Mercurial repository into T9548: Support Mercurial's bundle2 wire protocol.
Jan 9 2017, 3:14 PM · Diffusion, Mercurial
epriestley merged task T10382: Transaction abort when pushing via SSH to a Mercurial repository into T9548: Support Mercurial's bundle2 wire protocol.
Jan 9 2017, 3:14 PM · Mercurial, Bug Report
epriestley added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

All evidence points to this and T9548 being the same task, right? I'm just going to merge them.

Jan 9 2017, 3:14 PM · Mercurial, Bug Report
epriestley closed T12083: Can't set Related Task/Commit as Resolved.
Jan 9 2017, 3:04 PM · Bug Report (Needs Information)
jeremy.norris added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

I believe there's more to it than just the bundle2 setting that causes it to not work. It still breaks for me even with devel.legacy.exchange=bundle1:

Jan 9 2017, 3:00 PM · Mercurial, Bug Report
OCram updated the question details for Q548: How to maintain uploaded files?.
Jan 9 2017, 2:18 PM · Files
OCram updated Q548: How to maintain uploaded files? from to How to maintain uploaded files?.
Jan 9 2017, 2:18 PM · Files
marvin.xf added a comment to T12083: Can't set Related Task/Commit.

I have restarted httpd. I try to disable some chrome extension. Now it works.

Jan 9 2017, 12:32 PM · Bug Report (Needs Information)
OCram edited the content of Community Resources.
Jan 9 2017, 10:46 AM
ralph.van.etten updated Q547: Stop tracking time for user who left. from to Stop tracking time for user who left..
Jan 9 2017, 9:03 AM · Phrequent
otto.bretz added a comment to T5523: Make bulk moves easier on Workboards.

Move all cards in this column to <another column>

That would be very useful for us as well.

Jan 9 2017, 8:06 AM · Restricted Project, Workboards
chad added a comment to T12083: Can't set Related Task/Commit.

I also can't reproduce this locally or on a Phacility test instance.

Jan 9 2017, 6:41 AM · Bug Report (Needs Information)
chad edited projects for T12083: Can't set Related Task/Commit, added: Bug Report (Needs Information); removed Bug Report.
Jan 9 2017, 6:35 AM · Bug Report (Needs Information)
chad added a comment to T12083: Can't set Related Task/Commit.

I can't reproduce the issue you describe following your steps on this server. Do you have any suggestions on what I could do differently to reproduce the bug?

Jan 9 2017, 6:35 AM · Bug Report (Needs Information)
marvin.xf added a comment to T12083: Can't set Related Task/Commit.

official website click related "Close As Duplicate", browser send two request.

Screen Shot 2017-01-09 at 14.23.20.png (229×948 px, 49 KB)

Jan 9 2017, 6:34 AM · Bug Report (Needs Information)
marvin.xf created T12083: Can't set Related Task/Commit.
Jan 9 2017, 6:17 AM · Bug Report (Needs Information)
chad added a revision to T5867: Move Quick Create to MenuItemEngine: Unknown Object (Differential Revision).
Jan 9 2017, 5:38 AM · Design
chad removed projects from T5867: Move Quick Create to MenuItemEngine: Phurl, Flags.
Jan 9 2017, 4:25 AM · Design
chad added a comment to T5867: Move Quick Create to MenuItemEngine.

Started building and insanely easier as an application. "Favorites" unless you have a better suggestion. Seems more flexible.

Jan 9 2017, 4:25 AM · Design
cspeckmim added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

For now, I'd suggest that if phabricator breaks bundle2, filter the response to the capabilities command and omit bundle2=.* from the response.

Jan 9 2017, 2:30 AM · Mercurial, Bug Report
cspeckmim added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

To be clear my suggestion was only for using 3.4 from client machines. Server machines can still be kept up to date.

Jan 9 2017, 2:04 AM · Mercurial, Bug Report
durin42 added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

Note that my suggestion in https://secure.phabricator.com/T10382#191728 is still what I think /should/ happen: if Phabricator insists on trying to parse the Mercurial protocol, it should absolutely not include capabilities it doesn't understand when queried by the client.

Jan 9 2017, 2:04 AM · Mercurial, Bug Report
indygreg added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

Running vanilla Mercurial 3.4 should not be recommended as all vanilla versions older than 3.7.3 are vulnerable to a trio of CVEs, the most likely exploitable being CVE-2016-3630. Instead, Mercurial <4.0 can be configured to disable bundle2 via the experimental.bundle2-exp=false config option and Mercurial >=4.0 can do the same via devel.legacy.exchange=bundle1. Both of these options are undocumented. I discourage their use. But as a workaround for Phabricator, they are preferred to exposing clients to known security bugs.

Jan 9 2017, 1:40 AM · Mercurial, Bug Report

Jan 8 2017

joshuaspence created T12082: Differential emails are rendering literal HTML tags.
Jan 8 2017, 10:33 PM · Differential, Bug Report
epriestley renamed T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted from Files does not serve HTTP "Content-Range" correctly if the underlying blob data is encrypted to Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.
Jan 8 2017, 9:42 PM · Files, Public & Media Relations, Bug Report
epriestley renamed T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted from Embedding mov doesn't work on secure.phabricator.com to Files does not serve HTTP "Content-Range" correctly if the underlying blob data is encrypted.
Jan 8 2017, 9:42 PM · Files, Public & Media Relations, Bug Report
epriestley added a revision to T12074: conduit API extensions for subprojects and milestones.: Unknown Object (Differential Revision).
Jan 8 2017, 8:57 PM · Feature Request
epriestley added a revision to T12074: conduit API extensions for subprojects and milestones.: Unknown Object (Differential Revision).
Jan 8 2017, 8:37 PM · Feature Request
epriestley added a revision to T12074: conduit API extensions for subprojects and milestones.: Unknown Object (Differential Revision).
Jan 8 2017, 8:21 PM · Feature Request
epriestley added a revision to T12074: conduit API extensions for subprojects and milestones.: Unknown Object (Differential Revision).
Jan 8 2017, 7:58 PM · Feature Request
boruchy awarded T4900: Workboards updating in real time a Love token.
Jan 8 2017, 7:57 PM · Workboards (v3), User Delight, Wikimedia
epriestley added a revision to T12074: conduit API extensions for subprojects and milestones.: Unknown Object (Differential Revision).
Jan 8 2017, 7:48 PM · Feature Request
epriestley added a revision to T12074: conduit API extensions for subprojects and milestones.: Unknown Object (Differential Revision).
Jan 8 2017, 7:32 PM · Feature Request
epriestley added a revision to T9058: PhabricatorPolicyFilter::hasCapability() does not apply application-level "Can Use Application" policy checks: Unknown Object (Differential Revision).
Jan 8 2017, 6:58 PM · Policy, Differential, Daemons
epriestley added a comment to T12074: conduit API extensions for subprojects and milestones..

After reviewing the state of the world in related tasks:

Jan 8 2017, 6:26 PM · Feature Request
• sshannin added a comment to T11992: daemon hits policy exception repeatedly.

Voila.

Jan 8 2017, 6:12 PM · Daemons, Policy, Bug Report
epriestley merged T5214: Add Conduit API methods for workboards into T12074: conduit API extensions for subprojects and milestones..
Jan 8 2017, 6:11 PM · Feature Request
epriestley merged task T5214: Add Conduit API methods for workboards into T12074: conduit API extensions for subprojects and milestones..
Jan 8 2017, 6:10 PM · Restricted Project, Conduit, Maniphest, Workboards
epriestley added a comment to T5214: Add Conduit API methods for workboards.

I'm going to merge this into T12074. Summarizing things so far and my short-term plans:

Jan 8 2017, 6:10 PM · Restricted Project, Conduit, Maniphest, Workboards
epriestley added a comment to T5793: Add support for personal workboards.

For T5214 / T12074, I am moving forward with an API query model where columns are associated with another, different object as a boardPHID (currently, always a project), and boards do not have their own objects or PHIDs.

Jan 8 2017, 6:02 PM · User Delight, Wikimedia, Workboards
epriestley added a comment to T11992: daemon hits policy exception repeatedly.

You can also safely cherry-pick that to test if it resolves things, it doesn't interact with anything else.

Jan 8 2017, 5:58 PM · Daemons, Policy, Bug Report
epriestley added a comment to T11992: daemon hits policy exception repeatedly.

That is: rP65c1c758ed261f65c8164d200ce01373ae5b822c

Jan 8 2017, 5:58 PM · Daemons, Policy, Bug Report
epriestley added a comment to T11992: daemon hits policy exception repeatedly.

If you upgrade past 65c1c758ed261f65c8164d200ce01373ae5b822c (both stable and master have this change), does it reproduce?

Jan 8 2017, 5:57 PM · Daemons, Policy, Bug Report
• sshannin added a comment to T11992: daemon hits policy exception repeatedly.

(And again, to emphasize, I'm happy to drop this if y'all aren't interested. I have my solution and am happy ignoring this, but just thought it might be something you guys would want to check out.)

Jan 8 2017, 5:57 PM · Daemons, Policy, Bug Report
• sshannin added a comment to T11992: daemon hits policy exception repeatedly.

Head as of my Dec. 17 post in this thread. Can roll forward, but was hoping to wait till the edit engine migration settled a bit.

Jan 8 2017, 5:55 PM · Daemons, Policy, Bug Report
epriestley added a comment to T11992: daemon hits policy exception repeatedly.

What version of Phabricator are you currently running?

Jan 8 2017, 5:51 PM · Daemons, Policy, Bug Report
• sshannin updated subscribers of T11992: daemon hits policy exception repeatedly.

@epriestley any suggestions?

Jan 8 2017, 5:49 PM · Daemons, Policy, Bug Report
epriestley added a comment to T12074: conduit API extensions for subprojects and milestones..

The *.search endpoints are preferred now, but the guidance we provide on the Conduit page could use some updating. I'll rework this a little alongside other changes here.

Jan 8 2017, 5:28 PM · Feature Request
epriestley added a comment to T12061: When problems occur in phd inside sudo error messages are neither printed nor logged.

I'm sorry you found this interaction frustrating. How would you prefer we have handled it?

Jan 8 2017, 4:51 PM · Feature Request
peerst added a comment to T12061: When problems occur in phd inside sudo error messages are neither printed nor logged.

Thanks for the warm welcome to the community

Jan 8 2017, 4:50 PM · Feature Request
epriestley edited projects for T12081: "Unable to establish a connection to any database host" via ssh git push, added: Bug Report (Needs Information); removed Bug Report.

The Phacility cluster is on 2017 Week 1 now and I can't reproduce this there, either:

Jan 8 2017, 4:28 PM · Bug Report (Needs Information)
epriestley closed T12080: Clicking "Manage Password" next to repo URI does not change page as Resolved by committing Restricted Diffusion Commit.
Jan 8 2017, 4:20 PM · Diffusion, Quicksand, Bug Report
epriestley added a comment to T12080: Clicking "Manage Password" next to repo URI does not change page.

Thanks for the report! We were just missing a "/" (Quicksand does not follow redirects, which I think is desirable, although maybe we'll change that eventually) but D17151 should fix this up a little more robustly.

Jan 8 2017, 4:02 PM · Diffusion, Quicksand, Bug Report
epriestley added a revision to T12080: Clicking "Manage Password" next to repo URI does not change page: Unknown Object (Differential Revision).
Jan 8 2017, 3:58 PM · Diffusion, Quicksand, Bug Report
epriestley added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

Specifically, here's what I believe the bug is, in rough terms that probably get some specifics wrong:

Jan 8 2017, 3:46 PM · Files, Public & Media Relations, Bug Report
epriestley added a comment to T12081: "Unable to establish a connection to any database host" via ssh git push.

This can either move forward as a bug report or a consulting request.

Jan 8 2017, 3:37 PM · Bug Report (Needs Information)
epriestley added a comment to T12061: When problems occur in phd inside sudo error messages are neither printed nor logged.

It was installed from a FreeBSD package

Jan 8 2017, 3:22 PM · Feature Request
zcourts updated the task description for T12081: "Unable to establish a connection to any database host" via ssh git push.
Jan 8 2017, 2:53 PM · Bug Report (Needs Information)
zcourts created T12081: "Unable to establish a connection to any database host" via ssh git push.
Jan 8 2017, 2:47 PM · Bug Report (Needs Information)
peerst added a comment to T12061: When problems occur in phd inside sudo error messages are neither printed nor logged.

Sorry for wasting your time. For the record:

Jan 8 2017, 12:43 PM · Feature Request
epriestley added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

.mov thing is likely an interaction between range requests and encryption. It works for me in Firefox. It only affects media files for installs which have configured file encryption and there's no data corruption/loss so I don't plan to get to it for a while unless a real install hits it.

Jan 8 2017, 7:12 AM · Files, Public & Media Relations, Bug Report
chad added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

ok forget it. works fine in other browser.

Jan 8 2017, 7:11 AM · Files, Public & Media Relations, Bug Report
chad added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

Chad's House ?

Jan 8 2017, 7:11 AM · Files, Public & Media Relations, Bug Report
epriestley added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

How do I reproduce that?

Jan 8 2017, 7:10 AM · Files, Public & Media Relations, Bug Report
chad updated subscribers of T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

@epriestley maybe also related? Some files are just missing in Conpherence now.

Jan 8 2017, 7:09 AM · Files, Public & Media Relations, Bug Report
chad updated the task description for T12080: Clicking "Manage Password" next to repo URI does not change page.
Jan 8 2017, 5:36 AM · Diffusion, Quicksand, Bug Report
Mnkras created T12080: Clicking "Manage Password" next to repo URI does not change page.
Jan 8 2017, 5:30 AM · Diffusion, Quicksand, Bug Report
cos added a comment to T12074: conduit API extensions for subprojects and milestones..

I am currently using project.query rather than project.search. I presume the change you suggest for .search would work for query too since my recollection is that project.query also doesn't return milestones. I tried to use .search originally, but noticed that the attachments available were limited and didn't correspond to the documentation (examples mentions projects, subscribers, but then later on only watchers and members are supported) so I was scared off by the 'may change' warning:-) I can switch to using .search if that's the better thing to do - I presume you'd add attachments for the new types of information to be returned?

Jan 8 2017, 2:27 AM · Feature Request
chad added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

My misunderstanding on this btw, I assumed mov was an option, not a default.

Jan 8 2017, 2:04 AM · Files, Public & Media Relations, Bug Report
chad renamed T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted from Embedding mov doesn't work? to Embedding mov doesn't work on secure.phabricator.com.
Jan 8 2017, 1:47 AM · Files, Public & Media Relations, Bug Report
chad added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

Anywho, I can't reproduce this in a local sandbox or on Phacility, so assume something is hokey with how files are served maybe in secure.

Jan 8 2017, 1:44 AM · Files, Public & Media Relations, Bug Report
chad added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

My expectation per Contributing Bug Reports is people submit bug reports they see on their real install. Otherwise we have no idea now to prioritize if something happened here testing vs. something people really hit in day to day use. Filing it here also means an upstream developer must respond to it.

Jan 8 2017, 1:15 AM · Files, Public & Media Relations, Bug Report
Mnkras added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

I expect it to work, because it looks like it is supposed to:

pasted_file (606×1 px, 89 KB)

Jan 8 2017, 12:58 AM · Files, Public & Media Relations, Bug Report
chad added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

And I think most people can't play them.

Jan 8 2017, 12:57 AM · Files, Public & Media Relations, Bug Report
chad added a comment to T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.

Why do you expect mov's to work? To my knowledge, this server isn't configured to support them.

Jan 8 2017, 12:56 AM · Files, Public & Media Relations, Bug Report
epriestley merged T12078: Toolbar shows I have notifications when I have none. into T8953: Phantom unread notifications when the user is subscribed to something he's not allowed to view.
Jan 8 2017, 12:37 AM · Wikimedia, Notifications
epriestley merged T12078: Toolbar shows I have notifications when I have none. into T8953: Phantom unread notifications when the user is subscribed to something he's not allowed to view.
Jan 8 2017, 12:37 AM · Wikimedia, Notifications
epriestley merged task T12078: Toolbar shows I have notifications when I have none. into T8953: Phantom unread notifications when the user is subscribed to something he's not allowed to view.
Jan 8 2017, 12:37 AM · Notifications, Bug Report
epriestley merged task T12078: Toolbar shows I have notifications when I have none. into T8953: Phantom unread notifications when the user is subscribed to something he's not allowed to view.
Jan 8 2017, 12:37 AM · Notifications, Bug Report
Mnkras created T12079: Files does not serve HTTP "Content-Range" requests correctly if the underlying blob data is encrypted.
Jan 8 2017, 12:36 AM · Files, Public & Media Relations, Bug Report
Mnkras created T12078: Toolbar shows I have notifications when I have none..
Jan 8 2017, 12:33 AM · Notifications, Bug Report

Jan 7 2017

• sshannin added a comment to T11992: daemon hits policy exception repeatedly.

I did some further digging and made a little progress

Jan 7 2017, 6:06 PM · Daemons, Policy, Bug Report
epriestley added a comment to T7899: DifferentialDiff->description is "text255?" but should be "text".

Yeah, this is in stable, just didn't make the cut for individual mention in the changelog. I think it promoted in Week 52 and I just lumped it into "Differential got infrastructure changes".

Jan 7 2017, 6:17 AM · Differential, Mercurial, Arcanist
0 added a comment to E1330: All day test.

Sorry! Will do.

Jan 7 2017, 4:12 AM
chad added a comment to E1330: All day test.

Please use your own install or a Phacility instance to test Phabricator. I appreciate that you're finding bugs, but this is our real install, not a demo.

Jan 7 2017, 4:02 AM
cspeckmim added a comment to T7899: DifferentialDiff->description is "text255?" but should be "text".

Ah ok

Jan 7 2017, 3:39 AM · Differential, Mercurial, Arcanist
0 cancelled E1330: All day test.
Jan 7 2017, 3:23 AM
0 cancelled E1329: All day test.
Jan 7 2017, 3:23 AM