Page MenuHomePhabricator
Feed All Stories

Apr 25 2016

epriestley added a comment to T10866: Missing Information in Diffusion User Guide: Repo Hosting.

Oh, I think I have a handle on the first issue -- you can ignore my questions about that.

Apr 25 2016, 2:34 PM · Diffusion, Documentation, Bug Report
epriestley added a comment to T10866: Missing Information in Diffusion User Guide: Repo Hosting.

It turned out that the git-user (aka vcs-user) created files as phd:phd

Apr 25 2016, 2:31 PM · Diffusion, Documentation, Bug Report
chad edited this Badge.
Apr 25 2016, 2:31 PM
florianb backed I2 Donate to Phacility with $25.00 USD.
Apr 25 2016, 2:25 PM · Phacility
florianb awarded Blog Post: Development Notes (2016 Week 10) a Piece of Eight token.
Apr 25 2016, 2:22 PM
florianb created T10866: Missing Information in Diffusion User Guide: Repo Hosting.
Apr 25 2016, 2:19 PM · Diffusion, Documentation, Bug Report
epriestley added a revision to T4292: Implement repository replication: Unknown Object (Differential Revision).
Apr 25 2016, 1:38 PM · Restricted Project, Diffusion
epriestley added a comment to T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too.

This also reproduced locally for me (OS X Yosemite, openssh-6.7p1), and chmod on the parent directory alone was reliably sufficient to prevent SSH from working. The instructions already suggest a root-owned location, but D15794 notes this requirement explicitly.

Apr 25 2016, 12:42 PM · Diffusion, Documentation
epriestley added a revision to T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too: Unknown Object (Differential Revision).
Apr 25 2016, 12:40 PM · Diffusion, Documentation
epriestley claimed T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too.
Apr 25 2016, 12:33 PM · Diffusion, Documentation
epriestley added a comment to T5463: Could not stat AuthorizedKeysCommand .

I can confirm T9560 locally, at least, so that might be the problem or part of it -- maybe start there? If you can provide details we're happy to fix this properly.

Apr 25 2016, 12:32 PM · Diffusion
florianb added a comment to T5463: Could not stat AuthorizedKeysCommand .

Thanks @epriestley, i will try to investigate further. It seems that i have still something misconfigured. I'll get back as soon as i have to add something useful. :-)

Apr 25 2016, 12:31 PM · Diffusion
epriestley added a comment to T5463: Could not stat AuthorizedKeysCommand .

Yes, you should NOT set AuthorizedKeysCommandUser root, this is needlessly dangerous and gives the script substantially more permission than it needs.

Apr 25 2016, 12:28 PM · Diffusion
florianb added a comment to T5463: Could not stat AuthorizedKeysCommand .

I ran into the same issue, it turned out, that the permissions for /phabricator, /libphutil and /phabricator as well as for the preceding parent-directory had to be 755.

Apr 25 2016, 12:21 PM · Diffusion
epriestley added a revision to T10865: Remarkup is not rendered in Pholio Application: Unknown Object (Differential Revision).
Apr 25 2016, 12:12 PM · Remarkup, Pholio, Bug Report
epriestley claimed T10865: Remarkup is not rendered in Pholio Application.
Apr 25 2016, 11:58 AM · Remarkup, Pholio, Bug Report
epriestley added a comment to T10865: Remarkup is not rendered in Pholio Application.

I also caught this vaguely related downstream report of markup list styles not rendering properly in mock descriptions:

Apr 25 2016, 11:58 AM · Remarkup, Pholio, Bug Report
sascha-egerer created T10865: Remarkup is not rendered in Pholio Application.
Apr 25 2016, 10:53 AM · Remarkup, Pholio, Bug Report
c.mertins awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Apr 25 2016, 6:40 AM · Workboards (v3), Restricted Project

Apr 24 2016

chad added a comment to T10864: A Rearkup/Text Paste should render with rmarkup rules.

Your request doesn't describe a problem in using Phabricator. See Contributing Feature Requests and Describing Root Problems. Specifically we don't see "feature doesn't exist" as a root problem. We need to know how you're using Phabricator and what problem you ran into using it. We do this so we can group similar problems together and solve them in a single way, and that may be significantly different than your suggested solution.

Apr 24 2016, 8:33 PM · Paste, Feature Request
• aHa added a comment to T10864: A Rearkup/Text Paste should render with rmarkup rules.

Sorry I read it, but I don't understand what is missing

Apr 24 2016, 8:19 PM · Paste, Feature Request
chad closed T10864: A Rearkup/Text Paste should render with rmarkup rules as Invalid.

Please see Contributing Feature Requests for what information is required for filing a new feature request.

Apr 24 2016, 8:06 PM · Paste, Feature Request
• aHa created T10864: A Rearkup/Text Paste should render with rmarkup rules.
Apr 24 2016, 7:56 PM · Paste, Feature Request
epriestley lowered the priority of T10860: After an inconsistent cluster repository write, consider just ignoring the lock from Normal to Low.

After we receive a push, try very very very very hard to release the lock if we're the thread holding it, even if we lost the global lock

Apr 24 2016, 6:39 PM · Clusters, Diffusion
epriestley closed T10863: Aphlict fails to start when "logs" option is omitted as Resolved by committing Restricted Diffusion Commit.
Apr 24 2016, 6:20 PM · Aphlict, Bug Report
epriestley claimed T10863: Aphlict fails to start when "logs" option is omitted.
Apr 24 2016, 6:20 PM · Aphlict, Bug Report
epriestley added a revision to T10860: After an inconsistent cluster repository write, consider just ignoring the lock: Unknown Object (Differential Revision).
Apr 24 2016, 5:59 PM · Clusters, Diffusion
epriestley added a revision to T10860: After an inconsistent cluster repository write, consider just ignoring the lock: Unknown Object (Differential Revision).
Apr 24 2016, 4:58 PM · Clusters, Diffusion
epriestley added a revision to T10860: After an inconsistent cluster repository write, consider just ignoring the lock: Unknown Object (Differential Revision).
Apr 24 2016, 4:07 PM · Clusters, Diffusion
epriestley added a revision to T10860: After an inconsistent cluster repository write, consider just ignoring the lock: Unknown Object (Differential Revision).
Apr 24 2016, 3:33 PM · Clusters, Diffusion
epriestley added a revision to T10863: Aphlict fails to start when "logs" option is omitted: Unknown Object (Differential Revision).
Apr 24 2016, 1:35 PM · Aphlict, Bug Report
epriestley closed T10857: Unexisting `protocol` option in Aphlict configuration mentioned in documentation as Resolved by committing Restricted Diffusion Commit.
Apr 24 2016, 1:32 PM · Aphlict, Documentation, Bug Report
hach-que created T10863: Aphlict fails to start when "logs" option is omitted.
Apr 24 2016, 7:48 AM · Aphlict, Bug Report

Apr 23 2016

chad updated the image for Security from F76097: profile-project.png to F1070088: fa-tags-red.png.
Apr 23 2016, 6:34 PM
avivey added a watcher for Security: avivey.
Apr 23 2016, 6:09 PM
epriestley added a comment to T10832: Evaluate Git remote execution vulnerabilities with 2GB pathnames.

Any test we execute must be distributed with phabricator/ and run on the first page load after an upgrade. That generally means it needs to be small, very fast, and completely conclusive. Compare to the test for Shellshock here, which runs in a few milliseconds, requires 10 lines of code, and is totally conclusive:

Apr 23 2016, 5:18 PM · Ops, Git, Phacility, Security
• ytrezq added a comment to T10832: Evaluate Git remote execution vulnerabilities with 2GB pathnames.

Broadly, I lean toward this policy going forward:

  • When we can perform an accurate test for the vulnerability in a reasonable amount of time/effort and tell you that you are definitely vulnerable (as with Shellshock), we will continue to do so with an active setup warning.
  • When we can not perform such a test (as here), we will publish guidance and note the issue in the changelog, but will not attempt to guess if the installed version may be vulnerable because this test will frequently be confusing/misleading/wrong.
  • We can re-evaluate this after T5055, which may give us a wider range of tools for providing more accurate vulnerability notifications.
Apr 23 2016, 4:53 PM · Ops, Git, Phacility, Security
• sshannin added a comment to T10833: arc which broken with subversion.

Well now I feel silly for not noticing that. I searched for the line number of the error ><

Apr 23 2016, 2:46 PM · FreeBSD, Restricted Project, Subversion, Arcanist, Bug Report
ofbeaton awarded T7259: Case insensitive URLs a Like token.
Apr 23 2016, 2:45 PM · Restricted Project, Aphront
chad merged task T10862: allow lowercase url item leading letters T305 vs t305 into T7259: Case insensitive URLs.
Apr 23 2016, 2:44 PM · Feature Request
chad merged T10862: allow lowercase url item leading letters T305 vs t305 into T7259: Case insensitive URLs.
Apr 23 2016, 2:44 PM · Restricted Project, Aphront
ofbeaton added a comment to T10862: allow lowercase url item leading letters T305 vs t305.

Even if it was just a redirect, that would be great.

Apr 23 2016, 2:39 PM · Feature Request
ofbeaton created T10862: allow lowercase url item leading letters T305 vs t305.
Apr 23 2016, 2:38 PM · Feature Request
epriestley merged task T10833: arc which broken with subversion into T6635: The `arc which` command triggers notices.
Apr 23 2016, 2:33 PM · FreeBSD, Restricted Project, Subversion, Arcanist, Bug Report
epriestley merged T10833: arc which broken with subversion into T6635: The `arc which` command triggers notices.
Apr 23 2016, 2:33 PM · Subversion, Arcanist
chad edited projects for T10859: Wiki page title is not updated in preview, added: Feature Request; removed Bug Report.
Apr 23 2016, 2:06 PM · Feature Request, Phriction
epriestley triaged T10859: Wiki page title is not updated in preview as Wishlist priority.

We'll fix this eventually, but probably not until the next time we do major work on Phriction (T9379) since the impact is very minor.

Apr 23 2016, 1:44 PM · Feature Request, Phriction
epriestley created T10861: Provide a tool to rewind the push log for a repository.
Apr 23 2016, 1:42 PM · Diffusion
epriestley created T10860: After an inconsistent cluster repository write, consider just ignoring the lock.
Apr 23 2016, 1:06 PM · Clusters, Diffusion
hyOzd created T10859: Wiki page title is not updated in preview.
Apr 23 2016, 12:54 PM · Feature Request, Phriction
epriestley added Q377: Is there a right way to define edit engines with separate "create" and "edit" modes? (Answer 367).
Apr 23 2016, 12:38 PM
• pmoreau added a comment to T10857: Unexisting `protocol` option in Aphlict configuration mentioned in documentation.

Ah, it makes sense. Thanks!

Apr 23 2016, 12:23 PM · Aphlict, Documentation, Bug Report
epriestley closed T10858: Wrong path to the `Notification Servers` status page in the documentation as Resolved.
Apr 23 2016, 12:22 PM · Documentation, Aphlict, Bug Report
• pmoreau added a comment to T10858: Wrong path to the `Notification Servers` status page in the documentation.

I am seeing the same, and it makes sense, once you know it only contains waypoints that you click.

Apr 23 2016, 12:21 PM · Documentation, Aphlict, Bug Report
epriestley claimed T10857: Unexisting `protocol` option in Aphlict configuration mentioned in documentation.
Apr 23 2016, 12:18 PM · Aphlict, Documentation, Bug Report
epriestley added a comment to T10857: Unexisting `protocol` option in Aphlict configuration mentioned in documentation.

Thanks! The intended instruction was just "don't specify ssl.key, ssl.cert, etc.". D15787 should fix it.

Apr 23 2016, 12:18 PM · Aphlict, Documentation, Bug Report
epriestley added a revision to T10857: Unexisting `protocol` option in Aphlict configuration mentioned in documentation: Unknown Object (Differential Revision).
Apr 23 2016, 12:16 PM · Aphlict, Documentation, Bug Report
epriestley added a comment to T10858: Wrong path to the `Notification Servers` status page in the documentation.

I don't have to click "Cluster", it's just a subheader. Are you seeing something different?

Apr 23 2016, 11:33 AM · Documentation, Aphlict, Bug Report
• pmoreau created T10858: Wrong path to the `Notification Servers` status page in the documentation.
Apr 23 2016, 11:16 AM · Documentation, Aphlict, Bug Report
• pmoreau created T10857: Unexisting `protocol` option in Aphlict configuration mentioned in documentation.
Apr 23 2016, 11:07 AM · Aphlict, Documentation, Bug Report
cinek awarded T10855: Allow external systems to edit the content of dashboard panels so they can publish information a Piece of Eight token.
Apr 23 2016, 9:51 AM · Dashboards, Restricted Project, Conduit, Feature Request
yaserkh awarded T8783: Unprototype Nuance (v1) a Love token.
Apr 23 2016, 7:50 AM · Nuance, Restricted Project
yelirekim added a comment to Q377: Is there a right way to define edit engines with separate "create" and "edit" modes?.

There is no trickery involved in our repository edit field, it might as well be a datasource field.

Apr 23 2016, 3:29 AM · ApplicationEditor
yelirekim updated Q377: Is there a right way to define edit engines with separate "create" and "edit" modes? from to Is there a right way to define edit engines with separate "create" and "edit" modes?.
Apr 23 2016, 3:26 AM · ApplicationEditor
epriestley edited the content of Changelog.
Apr 23 2016, 12:09 AM
epriestley created 2016 Week 17 (Very Late April).
Apr 23 2016, 12:09 AM

Apr 22 2016

joshuaspence added a comment to T9640: Make Phabricator compatible with PHP7.

FWIW I have updated to Ubuntu 16.04 locally and haven't had any issues with Arcanist.

Apr 22 2016, 11:05 PM · Prioritized, Restricted Project, Infrastructure
20after4 added a comment to Blog Post: Advanced Internet Usage (Experts ONLY!).

But for even more elite productivity, all web pages should implement command interpreters so we can have CLIs in our web pages.

Apr 22 2016, 8:38 PM
eadler added a member for Haskell.org: eadler.
Apr 22 2016, 8:37 PM
thoughtpolice added a member for Haskell.org: thoughtpolice.
Apr 22 2016, 8:30 PM
thoughtpolice created Haskell.org.
Apr 22 2016, 8:29 PM
20after4 added a comment to Blog Post: Advanced Internet Usage (Experts ONLY!).

This is almost exactly how I use the internets. Did I win at interneting?

Apr 22 2016, 8:29 PM
chad added a comment to T9640: Make Phabricator compatible with PHP7.

I'll fire up a VM this weekend and see if there are any issues.

Apr 22 2016, 6:43 PM · Prioritized, Restricted Project, Infrastructure
• kaendfinger added a comment to T9640: Make Phabricator compatible with PHP7.

@epriestley I misstated that. My intention was to say that Ubuntu is a platform which a decent chunk of people use. I definitely don't have data to back that up for Phabricator (And I don't think you guys have telemetry to be able to tell either).

Apr 22 2016, 6:41 PM · Prioritized, Restricted Project, Infrastructure
avivey created T10856: Small Revision page enhanchments.
Apr 22 2016, 6:02 PM · Restricted Project, Feature Request, Differential
chad added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

Yeah in those cases, writing your own "Internal Tools Status" application would probably be better.

Apr 22 2016, 5:55 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

A few things that might motivate writing an API panel (which pulls from external sources) instead:

Apr 22 2016, 5:53 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

(I agree that this is a bad solution for notifying users of downtime/maintenance on the current instance, and that rooster/broadcast is better.)

Apr 22 2016, 5:42 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

Maybe T8145 / T5231, but I think the driving use case there was "notify users of Phabricator downtime" specifically. I'm imagining the driving use cases here not affecting all users and not directly impacting Phabricator, just having wide enough impact that there's interest in surfacing them more broadly. e.g., "We're having a production incident affecting video upload, ops is investigating" is important, but not "every Phabricator user must be notified" important.

Apr 22 2016, 5:42 PM · Dashboards, Restricted Project, Conduit, Feature Request
chad added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

We also talked about a "Rooster" system as well. Something outside of dashboards that is tied to a time or is user-closable.

Apr 22 2016, 5:40 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

Doing a Phriction panel, an iframe panel, or a custom API panel would also be reasonable approaches.

Apr 22 2016, 5:39 PM · Dashboards, Restricted Project, Conduit, Feature Request
chad added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

I think we also have a "notify my users about event x or y or downtime" task floating around somewhere else.

Apr 22 2016, 5:39 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

Here's an example I could imagine us doing: we have some button on admin.phacility.com that says "Start a deploy". When that's pressed it causes these actions to occur:

Apr 22 2016, 5:38 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley renamed T10855: Allow external systems to edit the content of dashboard panels so they can publish information from Provide a dashboards conduit API to Allow external systems to edit the content of dashboard panels so they can publish information.
Apr 22 2016, 5:34 PM · Dashboards, Restricted Project, Conduit, Feature Request
chad added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

My concern is users will install their own dashboards and this doesn't particularly solve anything. I don't know what the core problem is to suggest an alternative solution.

Apr 22 2016, 5:32 PM · Dashboards, Restricted Project, Conduit, Feature Request
epriestley added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

This one seems OKish to me -- I think those use cases are pretty reasonable and root-problem-ish? I'm assuming "waht's currently broken" and "can we do deploys" are information known by some other system or tool.

Apr 22 2016, 5:30 PM · Dashboards, Restricted Project, Conduit, Feature Request
chad added a comment to T10855: Allow external systems to edit the content of dashboard panels so they can publish information.

Please see Contributing Feature Requests

Apr 22 2016, 5:20 PM · Dashboards, Restricted Project, Conduit, Feature Request
eadler created T10855: Allow external systems to edit the content of dashboard panels so they can publish information.
Apr 22 2016, 4:49 PM · Dashboards, Restricted Project, Conduit, Feature Request
chad closed T10854: Desktop notifications missing as Invalid.

Valid bug reports require steps to reproduce and version information. Please reopen the report once you've filled in this missing information. Thanks!

Apr 22 2016, 2:26 PM · Bug Report
• damio closed Q376: How to work with multiple branch to land as resolved.
Apr 22 2016, 2:22 PM · Differential, Arcanist
• damio updated the answer wiki for Q376: How to work with multiple branch to land.
Apr 22 2016, 2:22 PM · Differential, Arcanist
catalindazoot created T10854: Desktop notifications missing.
Apr 22 2016, 1:54 PM · Bug Report
epriestley added a project to T10853: Make `setCWD('/does/not/exist')` safer on ExecFuture: Infrastructure.
Apr 22 2016, 12:58 PM · Infrastructure
epriestley added a revision to T4292: Implement repository replication: Unknown Object (Differential Revision).
Apr 22 2016, 12:41 PM · Restricted Project, Diffusion
epriestley added a revision to T10853: Make `setCWD('/does/not/exist')` safer on ExecFuture: Unknown Object (Differential Revision).
Apr 22 2016, 12:26 PM · Infrastructure
epriestley created T10853: Make `setCWD('/does/not/exist')` safer on ExecFuture.
Apr 22 2016, 12:11 PM · Infrastructure
salvian added a comment to T7789: Support Git Large File Storage.

Thanks!
Yes, we will consider setting up alternate file domain

Apr 22 2016, 11:58 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a comment to T7789: Support Git Large File Storage.

I think D15784 should fix the issue. It should land in master later today and be available in stable in about 24 hours. Let me know if you're still seeing issues once you update past that.

Apr 22 2016, 11:47 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a revision to T7789: Support Git Large File Storage: Unknown Object (Differential Revision).
Apr 22 2016, 11:45 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a comment to T7789: Support Git Large File Storage.

I think I know what's going on, although I'm unsure why the <2MB files would work, unless you happened to push those to a public ("Visible To: Public (No Login Required)") repository and pushed the large files to a private repository?

Apr 22 2016, 11:17 AM · Customer Impact, Restricted Project, Files, Diffusion