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

Pgotsis refactor #32

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Pgotsis refactor #32

wants to merge 2 commits into from

Conversation

pgotsis
Copy link

@pgotsis pgotsis commented Dec 23, 2024

Complete refactor of the code. Basic changes:

  1. Split logic into functions.
  2. Change location of conf and logs based on root Vs. non-root user.
  3. Support installation of script in a standard location (such as /usr/local/bin) and still run it with logs from a standard user. Currently this is not possible as the script searches for the conf file in the script directory.
  4. Support of telegram topics.
  5. Check command dependencies.
  6. Use dig before nslookup for lookups.
  7. Add one more service for external IP detection

@pgotsis
Update update-cloudflare-dns.sh
575a165 
Complete refactor of the code. Basic changes:

1) Split logic into functions
2) Change location of conf and logs based on root Vs. non-root user.
3) Support installation of script in a standard location and still run it with logs from a standard user
4) Support of telegram topics
5) Check command dependencies
6) Use dig before nslookup for lookups
@pgotsis
Update update-cloudflare-dns.conf
ae119ff 
Changes to conf to support script refactoring
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pgotsis