这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@fwolfst
Copy link
Contributor

@fwolfst fwolfst commented Jul 16, 2015

In the line of #1303 , mention that parts of nginx-configuration can also be put in $APP/nginx.conf.d/ if default templates are used.

Reference to comment in other issue (does not look very nice in the documentation, can probably be removed).

@josegonzalez
Copy link
Member

This isn't dokku-alt, so this actually isn't true.

@fwolfst
Copy link
Contributor Author

fwolfst commented Jul 16, 2015

As far as I can tell from the distance (I do not use dokku-alt) the templates are the same (include **conf.d) and it works like a charm if I didn't mess my test up.

@michaelshobbs
Copy link
Member

@josegonzalez thoughts on adding this to the default template?

@fwolfst
Copy link
Contributor Author

fwolfst commented Jul 21, 2015

@michaelshobbs @josegonzalez It IS part of the default templates, its just not mentioned in the docs.

docs/nginx.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

I think you mean megabytes?

@josegonzalez
Copy link
Member

This can be merged once the two comments are fixed.

josegonzalez added a commit that referenced this pull request Jul 27, 2015
In doc: mention alternative to nginx.conf templates: include-dir.
@josegonzalez josegonzalez merged commit e9efcbe into dokku:master Jul 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants