[PR #32] [MERGED] Refactor entire Go codebase #584

New Issue

saavagebueno · 2025-11-20T04:25:20-05:00

saavagebueno commented

2025-11-20 04:25:20 -05:00

📋 Pull Request Information

Original PR: https://github.com/qdm12/ddns-updater/pull/32
Author: @qdm12
Created: 2/21/2020
Status: ✅ Merged
Merged: 2/22/2020
Merged by: @qdm12

Base: master ← Head: refactor

📝 Commits (10+)

f69e3a9 Total refactoring (untested)
a93b303 Fix UI templating
9d1e314 Fix parameters reading
fcf8c45 Fix sqlite t_last column kept
8e3caf1 Fix HTTP routing
6a70606 Fixed force update
45b11e4 GetDuration renamed to GetDelay
c0d860f Updated environment variables
f01c78a Fixed force update http handler
b362b29 Faster boot of web ui

📊 Changes

52 files changed (+1955 additions, -1694 deletions)

View changed files

📝 .vscode/settings.json (+1 -3)
📝 Dockerfile (+7 -4)
📝 README.md (+72 -75)
📝 ci.sh (+10 -2)
📝 cmd/updater/main.go (+74 -58)
📝 docker-compose.yml (+8 -5)
📝 go.mod (+1 -2)
📝 go.sum (+10 -14)
➕ internal/constants/ipmethod.go (+17 -0)
➕ internal/constants/providers.go (+26 -0)
➕ internal/constants/regex.go (+46 -0)
➕ internal/constants/status.go (+17 -0)
➕ internal/constants/url.go (+10 -0)
➕ internal/data/data.go (+40 -0)
➕ internal/data/memory.go (+32 -0)
➕ internal/data/persistence.go (+38 -0)
➖ internal/database/database.go (+0 -49)
➖ internal/database/queries.go (+0 -92)
📝 internal/env/env.go (+132 -89)
➕ internal/handlers/handlers.go (+64 -0)

...and 32 more files

📄 Description

Small UI adjustments
Do now show all previous IP addresses in notifications and UI
Database uses interfaces to be modular/pluggable in order to move away from sqlite
Less dependencies, even uses a switch instead of httprouter
Updated golibs
Changed default logging format to console (zap)
Better code overall, modular updater and trigger system
Refactored readme
CI script improved

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/qdm12/ddns-updater/pull/32 **Author:** [@qdm12](https://github.com/qdm12) **Created:** 2/21/2020 **Status:** ✅ Merged **Merged:** 2/22/2020 **Merged by:** [@qdm12](https://github.com/qdm12) **Base:** `master` ← **Head:** `refactor` --- ### 📝 Commits (10+) - [`f69e3a9`](https://github.com/qdm12/ddns-updater/commit/f69e3a95c1291d9ccc5d1b791683ee1f1e2e13c7) Total refactoring (untested) - [`a93b303`](https://github.com/qdm12/ddns-updater/commit/a93b303130b4f5a8be45fec0ac4c5dfc3a5d1db8) Fix UI templating - [`9d1e314`](https://github.com/qdm12/ddns-updater/commit/9d1e314f0337a05aeff6bea491e4c0f8aee9a494) Fix parameters reading - [`fcf8c45`](https://github.com/qdm12/ddns-updater/commit/fcf8c45006df90b20d177e4b832d80da3141113d) Fix sqlite t_last column kept - [`8e3caf1`](https://github.com/qdm12/ddns-updater/commit/8e3caf19b3fd552db39d2b4583cf8d1518ea68e0) Fix HTTP routing - [`6a70606`](https://github.com/qdm12/ddns-updater/commit/6a706065ce40355a871a805a682e4a0f65c6491e) Fixed force update - [`45b11e4`](https://github.com/qdm12/ddns-updater/commit/45b11e490ebd44f0a584ec58afb1b4e51015d407) GetDuration renamed to GetDelay - [`c0d860f`](https://github.com/qdm12/ddns-updater/commit/c0d860f537af1bf5fd36f77972fe0e332977472f) Updated environment variables - [`f01c78a`](https://github.com/qdm12/ddns-updater/commit/f01c78ae22fbc0750903f7aac5689790fbaaaad3) Fixed force update http handler - [`b362b29`](https://github.com/qdm12/ddns-updater/commit/b362b299ef2849602edcee82bce585a853ad520e) Faster boot of web ui ### 📊 Changes **52 files changed** (+1955 additions, -1694 deletions) <details> <summary>View changed files</summary> 📝 `.vscode/settings.json` (+1 -3) 📝 `Dockerfile` (+7 -4) 📝 `README.md` (+72 -75) 📝 `ci.sh` (+10 -2) 📝 `cmd/updater/main.go` (+74 -58) 📝 `docker-compose.yml` (+8 -5) 📝 `go.mod` (+1 -2) 📝 `go.sum` (+10 -14) ➕ `internal/constants/ipmethod.go` (+17 -0) ➕ `internal/constants/providers.go` (+26 -0) ➕ `internal/constants/regex.go` (+46 -0) ➕ `internal/constants/status.go` (+17 -0) ➕ `internal/constants/url.go` (+10 -0) ➕ `internal/data/data.go` (+40 -0) ➕ `internal/data/memory.go` (+32 -0) ➕ `internal/data/persistence.go` (+38 -0) ➖ `internal/database/database.go` (+0 -49) ➖ `internal/database/queries.go` (+0 -92) 📝 `internal/env/env.go` (+132 -89) ➕ `internal/handlers/handlers.go` (+64 -0) _...and 32 more files_ </details> ### 📄 Description - Small UI adjustments - Do now show *all* previous IP addresses in notifications and UI - Database uses interfaces to be modular/pluggable in order to move away from sqlite - Less dependencies, even uses a switch instead of httprouter - Updated golibs - Changed default logging format to `console` (zap) - Better code overall, modular updater and trigger system - Refactored readme - CI script improved --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

saavagebueno added the pull-request label 2025-11-20 04:25:20 -05:00

saavagebueno closed this issue

2025-11-20 04:25:20 -05:00

saavagebueno referenced this issue

2025-11-20 04:25:53 -05:00

[PR #584] chore(deps): bump docker/setup-qemu-action from 2 to 3 #737

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ddns-updater#584