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.

Sign up for GitHub

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

Jump to bottom

[debug] cloudflare如果配置的域名包含大写 cloudflare则不会删除旧的ip 而是直接新增一个相同子域名 #431

Closed
LOVE2CMOL opened this issue Mar 21, 2024 · 1 comment · Fixed by #433
Closed

[debug] cloudflare如果配置的域名包含大写 cloudflare则不会删除旧的ip 而是直接新增一个相同子域名 #431

LOVE2CMOL opened this issue Mar 21, 2024 · 1 comment · Fixed by #433
Labels
bug

Comments

@LOVE2CMOL
Copy link

cloudflare如果配置的域名包含大写 cloudflare则不会删除旧的ip 而是直接新增一个相同子域名

版本信息 (version info)

  • DDNS Version: v3.0.0@2024-03-01T11:59:30+00:00
  • OS Version： Linux version 5.14.0-284.30.1.el9_2.x86_64 ([email protected]) (gcc (GCC) 11.3.1 20221121 (Red Hat 11.3.1-4), GNU ld version 2.35.2-37.el9) Update ip.py #1 SMP PREEMPT_DYNAMIC Sat Sep 16 09:55:41 UTC 2023
  • Type(运行方式): Python3

复现步骤 (To Reproduce)

设置一个包含大小写的域名即可
image

补充说明 (Additional context)

我估计是代码中没有做大小写匹配 导致的问题
@LOVE2CMOL LOVE2CMOL added the bug label Mar 21, 2024
@NewFuture
Copy link
Owner

统一小写处理，可以fix这个bug

@laozhoubuluo laozhoubuluo mentioned this issue Apr 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(run): domain name uniformly lowercase to avoid problems laozhoubuluo/DDNS
2 participants
@NewFuture @LOVE2CMOL