Skip to content
This repository has been archived by the owner on Jul 30, 2019. It is now read-only.

Resolv host before checking WHM hash #102

Merged
merged 1 commit into from
Aug 12, 2015
Merged

Resolv host before checking WHM hash #102

merged 1 commit into from
Aug 12, 2015

Conversation

newx
Copy link
Contributor

@newx newx commented Aug 10, 2015

Sometimes the remote server is unreachable but it still fails with a message like "Missing WHM hash for x.x.x.x". This tries to resolve the host, and raises an exception if it fails, before checking the WHM hash.

Also improves error message when the host is down or unreachable.

@newx newx force-pushed the bugfix/resolve_error branch 2 times, most recently from ed8240d to d5497b8 Compare August 11, 2015 20:52
@@ -238,7 +238,7 @@ def transfer_tool

private

def do_request(uri, function, params)
def do_request(uri, function, params)
Copy link
Contributor

Choose a reason for hiding this comment

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

extra spaces

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@boone
Copy link
Contributor

boone commented Aug 12, 2015

👍

@newx newx force-pushed the bugfix/resolve_error branch 2 times, most recently from 3e9f33d to 15ab310 Compare August 12, 2015 15:14
boone added a commit that referenced this pull request Aug 12, 2015
Resolv host before checking WHM hash
@boone boone merged commit ea84209 into master Aug 12, 2015
@boone boone deleted the bugfix/resolve_error branch August 12, 2015 17:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants