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

Conversation

@d6rkaiz
Copy link
Contributor

@d6rkaiz d6rkaiz commented Jul 31, 2022

Since 0.27.9 include /home/dokku/nginx.conf.d/*.conf is duplicate defined.

# nginx -t
nginx: [emerg] "client_max_body_size" directive is duplicate in /home/dokku/default/nginx.conf.d/upload.conf:1
# head -n 15 nginx.conf

server {
  listen      [::]:80;
  listen      80;
  server_name example.com;
  access_log  /var/log/nginx/default-access.log;
  error_log   /var/log/nginx/default-error.log;

  include /home/dokku/default/nginx.conf.d/*.conf;
  location / {
    return 301 https://$host:443$request_uri;
  }

  include /home/dokku/default/nginx.conf.d/*.conf;
}

Closes #5277

@DavidLemayian
Copy link
Contributor

DavidLemayian commented Aug 1, 2022

This! Blocks setting up with letsencrypt

cc @josegonzalez

@josegonzalez
Copy link
Member

Wow! I'm re-running the failing tests.

@josegonzalez josegonzalez changed the title fix duplicate syntax for nginx.conf Remove duplicate syntax for nginx.conf Aug 1, 2022
@josegonzalez josegonzalez merged commit ef5a2a9 into dokku:master Aug 1, 2022
@d6rkaiz d6rkaiz deleted the fix_duplicate_nginxconf branch August 1, 2022 15:11
@d6rkaiz
Copy link
Contributor Author

d6rkaiz commented Aug 1, 2022

thanks!

github-actions bot pushed a commit that referenced this pull request Aug 1, 2022
# History

## 0.27.10

Install/update via the bootstrap script:

```shell
wget https://raw.githubusercontent.com/dokku/dokku/v0.27.10/bootstrap.sh
sudo DOKKU_TAG=v0.27.10 bash bootstrap.sh
```

### Bug Fixes

- #5276: @d6rkaiz Remove duplicate syntax for nginx.conf

### Other

- #5274: @kennell readme: mention Ubuntu 22.04 compability
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.

Nginx Template Duplicate include Entry

3 participants