Missing email

[Phelic2k]

This is an awesome project!
Right now I am getting an error when a new certificate is created:

simp_le: error: argument --email: expected one argument

Shouldn't the "DEFAULT_EMAIL" be enough here?

My docker-compose.yml:

version: '2'
services:
  frontproxy:
    build: ./frontproxy
    hostname: frontproxy
    restart: always
    container_name: frontproxy
    environment:
        DEFAULT_HOST: testvhost.local
        ENABLE_IPV6: "true"
        NETWORK_ACCESS: internal
    ports:
      - "80:80"
      - "443:443
    volumes:
      - /var/run/docker.sock:/tmp/docker.sock:ro
      - /etc/nginx/certs:/etc/nginx/certs:ro
      - /etc/nginx/conf.d:/etc/nginx/conf.d
      - /etc/nginx/dhparam:/etc/nginx/dhparam
      - /etc/nginx/vhost.d:/etc/nginx/vhost.d
      - /usr/share/nginx/html:/usr/share/nginx/html
  frontproxyssl:
    build: ./frontproxyssl
    container_name: frontproxyssl
    hostname: frontproxyssl
    restart: always
    depends_on:
      - frontproxy
    environment:
      - DEFAULT_EMAIL=user@domain.tld
    volumes_from:
      - frontproxy
    volumes:
      - /etc/nginx/certs:/etc/nginx/certs:rw
      - /var/run/docker.sock:/var/run/docker.sock:ro

The according Dockerfile:
FROM jrcs/letsencrypt-nginx-proxy-companion:latest

EDIT:
the environment variable LETSENCRYPT_EMAIL in my project overrode the setting

[buchdag]

That might be considered a bug, but I have no idea yet how we should treat *_EMAIL environment variables priority

[Phelic2k]

sorry, that was my fault.
Everything is working as expected. The project's settings should overrride the default.

[buchdag]

The project's settings should overrride the default.

That would make sense to me too, unfortunately that would also require a complete re-work of the way accounts are handled internally.

[buchdag]

@Phelic2k getting back to you on this one, if I understand the issue correctly you had an empty LETSENCRYPT_EMAIL environment variable in your project that overrode the DEFAULT_EMAIL ?