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

Updating 9/5/20 #1

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

Updating 9/5/20 #1

wants to merge 11 commits into from

Conversation

ToddKlindt
Copy link
Owner

No description provided.

ToddKlindt and others added 11 commits February 11, 2018 21:25
Fixed opening quote on the first line.
Modified port of GoDaddy_Powershell_DDNS.ps to bash script (.sh) posted on closed GoDaddy Forum by user "sanctus" on 3/19/2017. Thread is closed for new comments so can't update on there and thought it might be helpful for Linux users wanting to use your script without installing powershell. 

I modified the original port of the shell script provided by sanctus on GoDaddy forum to have proper syntax for operators as the one on the site fails in at least CentOs/RHEL 7 distributions on the if statement 


Original version posted on forum; (line 34)
if [ $dnsIp != $currentIp ]; 

Modified if statement; must  (line 34)
 if [ "$dnsIpa" = "$currentIp" ]; 

Original Forum URL to discussion (thread closed for comments so can't update nor reply to that post) 

https://www.godaddy.com/community/Managing-Domains/Dynamic-DNS-Updates/m-p/48744
Fixes by other contributors combined by Taegost.  Thanks for the updates.
Shell script request fix
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.

7 participants