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

Bug: Email address with + in username triggers invalid email format error. #112

Closed
3 of 23 tasks
hollajandro opened this issue Oct 21, 2020 · 1 comment
Closed
3 of 23 tasks
Assignees

Comments

@hollajandro
Copy link

TLDR: Email address with + on user side of address fails verification and triggers ERROR invalid email format error.

  1. Is this urgent?

    • Yes
    • No
  2. What DNS service provider(s) are you using?

    • Cloudflare
    • DDNSS.de
    • DonDominio
    • DNSOMatic
    • DNSPod
    • Dreamhost
    • DuckDNS
    • DynDNS
    • GoDaddy
    • Google
    • He.net
    • Infomaniak
    • Namecheap
    • NoIP
  3. What's the version of the program?

    See the line at the top of your logs

    Running version latest built on 2020-10-13T01:10:51Z (commit 860bc02)

  4. What are you using to run the container?

    • Docker run
    • Docker Compose
    • Kubernetes
    • Docker stack
    • Docker swarm
    • Podman
    • Other:
  5. Extra information

Logs:

Running version latest built on 2020-10-13T01:10:51Z (commit 860bc02)

🔧 Need help? https://github.com/qdm12/ddns-updater/issues/new
💻 Email? quentin.mcgaw@gmail.com
☕ Slack? Join from the Slack button on Github
💸 Help me? https://github.com/sponsors/qdm12
2020-10-20T21:26:33.038Z ERROR invalid email format

Configuration file (remove your credentials!):

{
"settings": [
{
"zone_identifier": "",
"provider": "cloudflare",
"domain": "example.com",
"host": "hostname",
"ttl": 1,
"email": "user+name@gmail.com",
"key": "",
"proxied": false,
"ip_version": "ipv6"
}
]
}

Host OS:

RancherOS 1.5.6

qdm12 added a commit that referenced this issue Oct 25, 2020
- All network requests are run with a context for faster shutdowns
- No node id variable for logging
- Stability improvements
@qdm12
Copy link
Owner

qdm12 commented Oct 25, 2020

Sorry for the delay; it should now be fixed if you re-pull the docker image

docker-compose pull
docker-compose up -d --force-recreate

Let me know if it still doesn't work. Thanks!

@qdm12 qdm12 closed this as completed Oct 25, 2020
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

No branches or pull requests

2 participants