Skip to content

Conversation

@bash-stack
Copy link

  • can update RR by type , does not enable debug by default
  • enabled updates for any record type

M°ŋα Ł¹ſα added 2 commits January 12, 2021 13:20
* loads includes from folder auf autodns-cli
* updated README
Copy link
Contributor

@till till left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies for the long time it took, I wasn't watching this repository.

I really appreciate you took the time to send a PR, would you be up for making the changes I am asking for? Please let me know! :)

### Updating/Creating a record
### Updating/Creating a DNS record ( resource record in autodns )

## BE CAREFUL AS the domain parameter might be prefixed ( e.g mydomainentry.mydomain.com)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
## BE CAREFUL AS the domain parameter might be prefixed ( e.g mydomainentry.mydomain.com)
> **BE CAREFUL** AS the domain parameter might be prefixed ( e.g mydomainentry.mydomain.com)


```
$ ./autodns-cli update foo.example.org 127.0.0.1
$ ./autodns-cli update domain.com 127.0.0.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure this was intended?

OR specify the type as last argument

```
$ ./autodns-cli update _amazonses.subdomain "ASDOIJQWDQWDASD+24hjdsf23/" TXT
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool!

declare -r ttl=120

if [ -f './.autodns-cli.rc' ]; then
if [ -f '~/.autodns-cli.rc' ]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you make it possibly load either? I am not sure what the best load order would be, maybe home first, then local second?

#source ./func.sh
#source ./commands.sh

source $( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )/func.sh
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼 Cheers!

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.

2 participants