Skip to content

hardcoded path for post-receive hook fs-exists check should not fatal gitlab:app:status

Created by: damianb

Considering that a multitude of people (including myself) frown upon the use of /home/ for serving anything out of, and instead prefer other locations within the filesystem for locating servers (such as /srv/), it should be expected that the post-receive hook may not exist there.

Throwing a fatal error when it does not exist there is useless in circumstances like my own.

A better alternative would be a warning, IMO - "We don't think this is right - check this unless you know what you're doing" would be far better than "File does not exist, return 1 and BAIL OUT"