Skip to content

Conversation

mrohnstock
Copy link
Contributor

Allow downloading attachments from gitlab.
Configure ProxyPass, to allow access to gitlab from anywhere.

DocumentRoot should lead to public-folder of gitlab.

Allow downloading attachments from gitlab.
Configure ProxyPass, to allow access to gitlab from anywhere.

DocumentRoot should lead to public-folder of gitlab.
@saschagrunert
Copy link

Great. Really really great.

@riyad
Copy link
Contributor

riyad commented Jan 23, 2013

+1

@axilleas
Copy link
Contributor

r+ but I think document root should match the one in the official installation guide, that is /home/git/gitlab/public.

@@ -8,12 +8,21 @@
#RewriteCond %{SERVER_PORT} ^80$
#RewriteRule ^(.*)$ https://%{SERVER_NAME}$1 [L,R]

ProxyPass /uploads !

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also added ProxyPass /error ! after this line to fix the display of HTTP errors coming from apache. Without this, apache tries to load its error page from e.g. /error/HTTP_BAD_GATEWAY.html.var but that gets proxied to unicorn, which then shows a 404 instead of the actual error.

@adaugherity
Copy link

+1; I added a note to the diff about also excluding /error from the proxy so apache HTTP errors are handled properly rather than ending up at unicorn's 404 page.

to documented public-directory of gitlab.
@mrohnstock
Copy link
Contributor Author

Thanks @adaugherity and @axilleas , updated the commit with your enhancements.

axilleas added a commit that referenced this pull request Aug 5, 2013
Beware that adding `ProxyPass /uploads !` would be a security issue,
since uploads are publicly available without any authentification by default.

See: https://github.com/gitlabhq/gitlabhq/issues/348#issuecomment-21682402
@axilleas
Copy link
Contributor

axilleas commented Aug 5, 2013

Thank you, implemented manually in above commit, as I am working on a new directory structure.

@axilleas axilleas closed this Aug 5, 2013
@axilleas axilleas mentioned this pull request Aug 5, 2013
14 tasks
axilleas added a commit that referenced this pull request Aug 5, 2013
Beware that adding `ProxyPass /uploads !` would be a security issue,
since uploads are publicly available without any authentification by default.

See: https://github.com/gitlabhq/gitlabhq/issues/348#issuecomment-21682402
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants