Skip to content

Fixed for new Login Mechinism #14

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ghost
Copy link

@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

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

Any idea when these are getting merged?

@ghost
Copy link
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

@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

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

@xMAC94x
Copy link

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