Skip to content

Fixed for new Login Mechinism#14

Open
ghost wants to merge 1 commit intoobserverss:masterfrom
brycegough:master
Open

Fixed for new Login Mechinism#14
ghost wants to merge 1 commit intoobserverss:masterfrom
brycegough:master

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 26, 2015

GoDaddy updated their login mechanism, hasn't been working since 24th August 2015, I have updated pygodaddy to allow logins to work again.

GoDaddy updated their login mechanism, I have updated pygodaddy to allow logins to work again. A bit of hacky code but at least it works!
@AndreasLoow
Copy link
Copy Markdown

This crashes with

Traceback (most recent call last):
  [...]
  File "[...]/pygodaddy/__init__.py", line 1, in <module>
    from .client import GoDaddyClient, GoDaddyAccount, LoginError
  File "[...]/pygodaddy/client.py", line 108
    loginUri = 'https://sso.godaddy.com/v1/?path=%2Fdefault.aspx&app=mya&regionsite=au&marketid=en-AU'
                                                                                                     ^
TabError: inconsistent use of tabs and spaces in indentation

for me (using 3.4.3). #13 works however.

@andresvidal
Copy link
Copy Markdown

Any idea when these are getting merged?

@ghost
Copy link
Copy Markdown
Author

ghost commented Aug 31, 2015

@AndreasLoow I've been using this code since I put it up here and it's working fine, might be the way I copied it to github from my terminal, however 2 other people said it was working for them so I'm not sure what went wrong for you. #13 is just my code that claneys pulled and made a few changes to, I just hadn't made a pull request.

@AndreasLoow
Copy link
Copy Markdown

@brycegough: The problem, as the stack trace in my previous comment says, is that you mix tabs and spaces when indenting your code. But #13 is better anway, imho, as it does not contain irrelevant changes.

@pretyman
Copy link
Copy Markdown

@AndreasLoow agreed, #13 seems to have less changes and does the job

@xMAC94x
Copy link
Copy Markdown

xMAC94x commented Oct 12, 2015

#13 worked for me too.
I installed it directly from claneys via

pip install --upgrade git+git://github.com/claneys/pygodaddy.git

and upgraded to python3 (but i dont know if this was necessary)

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.

4 participants