Hotfix for #2 : Errors when type is TXT #20

Merged
Firq merged 0 commits from hotfix/TXT into main 2022-11-27 17:52:19 +00:00
Firq commented 2022-11-27 17:09:49 +00:00 (Migrated from gitlab.neshweb.net)

Added errorhandling to prevent #2 from happening.

The code will now print out any dns record that doesn't match the type of A or AAAA. This will be investigated further

Also updated from PUT to PATCH because the request only changes necessary fields (needs practice testing, can be reverted)

Added errorhandling to prevent #2 from happening. The code will now print out any dns record that doesn't match the `type` of `A` or `AAAA`. This will be investigated further Also updated from `PUT` to `PATCH` because the request only changes necessary fields (needs practice testing, can be reverted)
Firq commented 2022-11-27 17:09:50 +00:00 (Migrated from gitlab.neshweb.net)

requested review from @Neshura

requested review from @Neshura
Firq commented 2022-11-27 17:09:50 +00:00 (Migrated from gitlab.neshweb.net)

assigned to @Firq

assigned to @Firq
Neshura commented 2022-11-27 17:35:41 +00:00 (Migrated from gitlab.neshweb.net)

added 1 commit

  • b8c74f2a - Fix due to bad push on branch feature/dockerize

Compare with previous version

added 1 commit <ul><li>b8c74f2a - Fix due to bad push on branch feature/dockerize</li></ul> [Compare with previous version](/Neshura/cloudflare-dns-updater/-/merge_requests/2/diffs?diff_id=36&start_sha=6a059e680692102f782ee3a84b6174068a843929)
Neshura commented 2022-11-27 17:36:34 +00:00 (Migrated from gitlab.neshweb.net)

added 1 commit

  • 66a85c32 - Fix due to bad push on branch feature/dockerize

Compare with previous version

added 1 commit <ul><li>66a85c32 - Fix due to bad push on branch feature/dockerize</li></ul> [Compare with previous version](/Neshura/cloudflare-dns-updater/-/merge_requests/2/diffs?diff_id=37&start_sha=b8c74f2a354d9191c1ba3a9f1911fa01dd05fdf5)
Neshura commented 2022-11-27 17:39:38 +00:00 (Migrated from gitlab.neshweb.net)

added 1 commit

  • aaa3da5c - Added ipv4 specification to bash script

Compare with previous version

added 1 commit <ul><li>aaa3da5c - Added ipv4 specification to bash script</li></ul> [Compare with previous version](/Neshura/cloudflare-dns-updater/-/merge_requests/2/diffs?diff_id=39&start_sha=66a85c320df86ae039dbb14782676b6484258d06)
Neshura commented 2022-11-27 17:42:53 +00:00 (Migrated from gitlab.neshweb.net)

added 1 commit

  • 536a9cea - Revert change from put to patch for now

Compare with previous version

added 1 commit <ul><li>536a9cea - Revert change from put to patch for now</li></ul> [Compare with previous version](/Neshura/cloudflare-dns-updater/-/merge_requests/2/diffs?diff_id=41&start_sha=aaa3da5cbb06ccf08b0202fa6e308414abb10ce3)
Neshura commented 2022-11-27 17:43:37 +00:00 (Migrated from gitlab.neshweb.net)

added 1 commit

  • 56e14400 - Revert "Revert change from put to patch for now"

Compare with previous version

added 1 commit <ul><li>56e14400 - Revert &quot;Revert change from put to patch for now&quot;</li></ul> [Compare with previous version](/Neshura/cloudflare-dns-updater/-/merge_requests/2/diffs?diff_id=43&start_sha=536a9ceaa14cd06e32bbe505b2eec49af537f317)
Neshura commented 2022-11-27 17:49:09 +00:00 (Migrated from gitlab.neshweb.net)

added 2 commits

Compare with previous version

added 2 commits <ul><li>61f952f0 - Added venv to gitignore</li><li>439716ed - Try Catch around the ip put</li></ul> [Compare with previous version](/Neshura/cloudflare-dns-updater/-/merge_requests/2/diffs?diff_id=45&start_sha=56e144005b6b9569fd7d168dfda06bc3f811661c)
Neshura commented 2022-11-27 17:50:10 +00:00 (Migrated from gitlab.neshweb.net)

added 1 commit

Compare with previous version

added 1 commit <ul><li>2b0dae76 - Uncaught syntax error</li></ul> [Compare with previous version](/Neshura/cloudflare-dns-updater/-/merge_requests/2/diffs?diff_id=46&start_sha=439716ed67756547999862e641632e89751e1ddf)
Neshura commented 2022-11-27 17:51:42 +00:00 (Migrated from gitlab.neshweb.net)

Had to make some changes to fix errors made by myself in earlier commits and unexpected errors in the python script

Had to make some changes to fix errors made by myself in earlier commits and unexpected errors in the python script
Neshura commented 2022-11-27 17:51:48 +00:00 (Migrated from gitlab.neshweb.net)

approved this merge request

approved this merge request
Neshura commented 2022-11-27 17:52:19 +00:00 (Migrated from gitlab.neshweb.net)

mentioned in commit fd19efd007

mentioned in commit fd19efd007e8c3803012cf7eec50c4c3c4d1e0b6
Neshura commented 2022-11-27 18:22:15 +00:00 (Migrated from gitlab.neshweb.net)

mentioned in issue #2

mentioned in issue #2
Neshura (Migrated from gitlab.neshweb.net) approved these changes 2023-12-10 00:57:58 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Neshura/cloudflare-dns-updater#20
No description provided.