[PR #3] [MERGED] Total code refactoring and added Dreamhost support #573

Closed
opened 2025-11-20 04:25:17 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/qdm12/ddns-updater/pull/3
Author: @qdm12
Created: 4/25/2019
Status: Merged
Merged: 4/25/2019
Merged by: @qdm12

Base: masterHead: dreamhost


📝 Commits (8)

  • 3bccebb Updated documentation
  • ad2aade Re-user same HTTP client for all requests
  • 2cfcfe6 Rework of main code and added sqlite database
  • a477f74 Removed debug log line
  • 7a53f69 Added Mutex for concurrent calls to Sqlite driver
  • b23152b http.Client is now in env variable; docker-compose updated to 3.1
  • ce88c48 Upgraded to Go 1.12.4 and updated readme
  • ed1823d Complete code refactoring

📊 Changes

44 files changed (+1934 additions, -1085 deletions)

View changed files

📝 .dockerignore (+7 -5)
📝 .gitignore (+0 -12)
📝 Dockerfile (+33 -23)
📝 README.md (+28 -15)
📝 docker-compose.yml (+4 -0)
go.mod (+15 -0)
go.sum (+60 -0)
main.go (+94 -0)
pkg/database/init.go (+36 -0)
pkg/database/queries.go (+80 -0)
pkg/healthcheck/query.go (+44 -0)
pkg/healthcheck/server.go (+68 -0)
pkg/logging/glogger.go (+91 -0)
pkg/logging/human.go (+53 -0)
pkg/logging/json.go (+75 -0)
pkg/logging/levels.go (+54 -0)
pkg/logging/logger.go (+95 -0)
pkg/models/history.go (+21 -0)
pkg/models/html.go (+45 -0)
pkg/models/recordconfig.go (+47 -0)

...and 24 more files

📄 Description

No description provided


🔄 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/3 **Author:** [@qdm12](https://github.com/qdm12) **Created:** 4/25/2019 **Status:** ✅ Merged **Merged:** 4/25/2019 **Merged by:** [@qdm12](https://github.com/qdm12) **Base:** `master` ← **Head:** `dreamhost` --- ### 📝 Commits (8) - [`3bccebb`](https://github.com/qdm12/ddns-updater/commit/3bccebb39311cd6f8d63272fcc8b972563f11bbe) Updated documentation - [`ad2aade`](https://github.com/qdm12/ddns-updater/commit/ad2aade27e787bd2da8340faf33847367a583023) Re-user same HTTP client for all requests - [`2cfcfe6`](https://github.com/qdm12/ddns-updater/commit/2cfcfe635a79b689e14b4904ad230aa72b5f5981) Rework of main code and added sqlite database - [`a477f74`](https://github.com/qdm12/ddns-updater/commit/a477f74babd3588a3587767df22bdef5ab8c9478) Removed debug log line - [`7a53f69`](https://github.com/qdm12/ddns-updater/commit/7a53f69d8c99ea6ce8ab6dcd88189bcafd934062) Added Mutex for concurrent calls to Sqlite driver - [`b23152b`](https://github.com/qdm12/ddns-updater/commit/b23152bdda67151a15e12922b341957825440b37) http.Client is now in env variable; docker-compose updated to 3.1 - [`ce88c48`](https://github.com/qdm12/ddns-updater/commit/ce88c48459bace2c60016c6559bc62b49bd6c9df) Upgraded to Go 1.12.4 and updated readme - [`ed1823d`](https://github.com/qdm12/ddns-updater/commit/ed1823d244eb314f81a8eea80926afd7872e91c9) Complete code refactoring ### 📊 Changes **44 files changed** (+1934 additions, -1085 deletions) <details> <summary>View changed files</summary> 📝 `.dockerignore` (+7 -5) 📝 `.gitignore` (+0 -12) 📝 `Dockerfile` (+33 -23) 📝 `README.md` (+28 -15) 📝 `docker-compose.yml` (+4 -0) ➕ `go.mod` (+15 -0) ➕ `go.sum` (+60 -0) ➕ `main.go` (+94 -0) ➕ `pkg/database/init.go` (+36 -0) ➕ `pkg/database/queries.go` (+80 -0) ➕ `pkg/healthcheck/query.go` (+44 -0) ➕ `pkg/healthcheck/server.go` (+68 -0) ➕ `pkg/logging/glogger.go` (+91 -0) ➕ `pkg/logging/human.go` (+53 -0) ➕ `pkg/logging/json.go` (+75 -0) ➕ `pkg/logging/levels.go` (+54 -0) ➕ `pkg/logging/logger.go` (+95 -0) ➕ `pkg/models/history.go` (+21 -0) ➕ `pkg/models/html.go` (+45 -0) ➕ `pkg/models/recordconfig.go` (+47 -0) _...and 24 more files_ </details> ### 📄 Description _No description provided_ --- <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:17 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ddns-updater#573