Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #18 -- adding an error page directive for custom error pages #20

Merged
merged 1 commit into from
Nov 24, 2016

Conversation

mechanicjay
Copy link

Adds an error page directive to each vhost. This allows custom 404 pages in the same way the official gitlab pages does: just add a 404.html to your project.

If you don't add a 404.html to your project, the behaviour is the same as today: a generic nginx 404 page.

@YuMS
Copy link
Owner

YuMS commented Nov 24, 2016

Need some tests and document for this.

@YuMS YuMS changed the base branch from master to 404-error-page November 24, 2016 05:07
@YuMS YuMS merged commit 1148083 into YuMS:404-error-page Nov 24, 2016
@YuMS
Copy link
Owner

YuMS commented Nov 24, 2016

After I created some tests and played around with this for a while. I found this solution won't always work because vhost config is only created after you specify a CNAME for a certain project.

For now, I can only make it possible for projects with CNAME to have customizable 404.html and leave a shared 404.html for the rest.

Any idea to solve this divergence?

@YuMS YuMS mentioned this pull request Nov 24, 2016
@YuMS YuMS mentioned this pull request Dec 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants