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.

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

All RR-Types , non-verbose #3

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

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)


Then run with:

```
$ ./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!

@@ -7,13 +7,16 @@ declare -r prog=$(basename "$0")
declare -r endpoint=https://api.autodns.com/v1
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