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

chore(deps): update cloudflare to 0.13 and remove wyn patch #64

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Pablo1107
Copy link

@Pablo1107 Pablo1107 commented Feb 5, 2025

Since cloudflare/cloudflare-rs#240 was merged upstream, we don't need the wyn patch, so update to 0.13 and remove the patch.

fixes #61

@Pablo1107 Pablo1107 changed the title Update cloudflare to 0.13 and remove wyn patch chore(deps): cloudflare to 0.13 and remove wyn patch Feb 5, 2025
@Pablo1107
Copy link
Author

This also needs to removed the locked, zone_id and zone_name from the codebase. locked can be removed trivially, but I don't think the same applies to the other two.

@Pablo1107 Pablo1107 force-pushed the update-cloudflare-rs-to-0.13 branch from 60e6a7c to 026ca96 Compare February 5, 2025 02:57
@Pablo1107 Pablo1107 force-pushed the update-cloudflare-rs-to-0.13 branch from 026ca96 to 74665d7 Compare February 5, 2025 03:02
@Pablo1107
Copy link
Author

Got it working, please review when available.

@Pablo1107 Pablo1107 changed the title chore(deps): cloudflare to 0.13 and remove wyn patch chore(deps): update cloudflare to 0.13 and remove wyn patch Feb 5, 2025
@Pablo1107
Copy link
Author

I don't know if cloudflare is moving too fast with their API changes but after Feb 05 21:40 UTC it started to break again even with this changes:

Feb 05 18:35:00 nixos systemd[1]: Started CloudFlare Dynamic DNS Client.
Feb 05 18:35:00 nixos cfdyndns-start[2984606]:  INFO  cfdyndns::ip > [redacted]
Feb 05 18:35:03 nixos cfdyndns-start[2984606]:  INFO  cfdyndns::dns > skipping A record `[redacted]`; already up to date
Feb 05 18:35:03 nixos cfdyndns-start[2984606]:  INFO  cfdyndns::dns > skipping A record `[redacted]`; already up to date
Feb 05 18:35:03 nixos systemd[1]: cfdyndns.service: Deactivated successfully.
Feb 05 18:35:03 nixos systemd[1]: cfdyndns.service: Consumed 65ms CPU time, 6M memory peak, 14K incoming IP traffic, 3.2K outgoing IP traf>
Feb 05 18:40:00 nixos systemd[1]: Started CloudFlare Dynamic DNS Client.
Feb 05 18:40:00 nixos cfdyndns-start[2990251]:  INFO  cfdyndns::ip > [redacted]
Feb 05 18:40:03 nixos cfdyndns-start[2990251]: Error: error decoding response body
Feb 05 18:40:03 nixos systemd[1]: cfdyndns.service: Main process exited, code=exited, status=1/FAILURE
Feb 05 18:40:03 nixos systemd[1]: cfdyndns.service: Failed with result 'exit-code'.

Every request before that went fine, every request after this shows the same error.

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

Successfully merging this pull request may close these issues.

Error: error decoding response body: missing field locked at line 1 column 433
2 participants