How did you install PHP, exactly?

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.

Ah, it makes sense. Thanks!

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

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

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

so would you mind closing the task? I can't do it myself.

I have no idea if we plan to add it or not. See Planning for how we prioritize time.

If you don't plan to add it: just reject the task - no problem. I just wanted to let you guys no that there is a slightly more secure way of doing what you provide to all the novice users out there.

Accepting contributions isn't free to us. We bear the cost of testing, accepting, documenting, and providing lifetime support. See https://secure.phabricator.com/book/phabcontrib/article/contributing_code/ for more details.

this still applies 2 years after sending a diff. is there missing something?

This is happening generally as part of T10751. I'm just going to close this out since I don't think it's useful as a separate task.

I plan to separate this configuration and make these options all explicit, see T10697.

For reference and the benefit of people Googling how to configure a script and regex linter, here is a full example (using sass-lint):

Importing SVN repositories as single repositories is recommended (Facebook's had several million commits in 2011, IIRC).

If there aren't docs yet, anyone willing to speak quickly on what elasticsearch integration entails?

Thanks!! its worked now.

@epriestley I have noticed that, in your first comment above, two links, titled "in this document", are broken (links are located in sections "Branching" and "Merging"). It would be nice to have those links to presumably useful and important documents fixed.

Thanks for this, just got caught by it myself.

(This is the 3rd time this came up in IRC). Looks like people are trying to keep phabricator-ssh-hook.sh close to the phabricator code.

(I think we're happy with the feature-space here)

@sgrimm In your large post, you have a link that says 'Add a Herald Rule'. This link appears to be intended as a template so you can replace the domain with your own domain.However, clicking it actually brings you to the working domain http://phabricator.xxx.com. The result is funny and unexpected, but likely not intended. Maybe it should be changed to phabricator.example.com to prevent people from being linked to porn :)

👍

Yeah, completeness is definitely sufficient motivation if you're up for it.

I've also noticed that Herald Transactions have PHID, but not PHIDType class (https://secure.phabricator.com/diffusion/P/browse/master/src/applications/herald/storage/transcript/HeraldTranscript.php;3f439e25bc586ff8bae203a47625f6c360e4402a$193). Wasn't sure if "completeness" is a good enough reason to make the changes (PHIDType + teach the Query object about withPHIDs, and maybe a bit more).

Tru dat.

I think I'll have time for this this week/weekend.

Haha, okay. Let's just add a link to the transcript.

I guess the short of it is, I have rules to allow pushes, instead of rules to block pushes:

Ah, your underlying issue is difficulty understanding why pushes are rejected?

Should we maybe have the dragon also put a link to the relevant herald transcript?

First: HeraldTestConsoleController is currently a big mess with a huge instanceof block.

Actually, that report is already fixed.

Actually, that report is already fixed. For completeness, the behavior adapters should have is:

See also T7961, which is another modular behavior adapters should have.

Related, the test console has a big if ($x instanceof ...) mess that should be pushed down into Adapters. A reasonable flow might be:

See also T9640 for tracking PHP7 compatibility.

I changed my configuration to be able to see the message again, so here is the screen shot: