In contrast to what is stated in D8719, the feature T182 works well
for us after applying this trivial change. Without this change, we
constantly had the same issue as
https://github.com/phacility/phabricator/issues/566
The error appears for me on a git repo hosted over http, while the
creator of that issue experiences the problem with a ssh-served
repo.
The root cause seems to be the following in phabricator/bin/commit-hook:
$username = getenv(DiffusionCommitHookEngine::ENV_USER);
if (!strlen($username)) {
throw new Exception(
pht('usage: %s should be defined!', DiffusionCommitHookEngine::ENV_USER));
}