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

GoDaddy throttling #393

Closed
feens opened this issue Mar 12, 2015 · 8 comments
Closed

GoDaddy throttling #393

feens opened this issue Mar 12, 2015 · 8 comments
Assignees

Comments

@feens
Copy link

feens commented Mar 12, 2015

I've noticed that GoDaddy throttles requests, but the parser has no support for it. Thus far I've added this method to extend the parser:

    def response_throttled?
      !!(content_for_scanner =~ /For complete domain details go to/)
    end
@weppos
Copy link
Owner

weppos commented Mar 12, 2015

That is not a throttled response, rather a partial response. The way GoDaddy throttles requests is by dropping the socket connection.

@weppos weppos closed this as completed Mar 12, 2015
@weppos weppos self-assigned this Mar 12, 2015
@weppos weppos added the wontfix label Mar 12, 2015
@feens
Copy link
Author

feens commented Mar 12, 2015

So what's the proper way of handling this then?

@weppos
Copy link
Owner

weppos commented Mar 12, 2015

There was some discussion to improve the ability to deal with separate parts in a thin request flow (#186), but it was never implemented.

@feens
Copy link
Author

feens commented Mar 12, 2015

Also...would that method be valid if changed to response_incomplete?

@weppos
Copy link
Owner

weppos commented Mar 12, 2015

Not really. response_incomplete was designed to deal with a partial response, generally caused by a network issue. In this case, it's just a limited content (some registrars limit content based on the number of requests to avoid data abuse). The response itself it's still formally valid and complete.

@feens
Copy link
Author

feens commented Mar 12, 2015

It may be valid, but at least in the case, if I try and query expires_on, I get nil, even though if I'm looking at the text whois, there is an expires_on provided. Maybe the issue isn't in the part-parser itself then, but that how it's decided on which part-parser to use.

@weppos
Copy link
Owner

weppos commented Mar 12, 2015

It should be falling back to the parent parser, if the specific one doesn't contain the value.

@feens
Copy link
Author

feens commented Mar 12, 2015

well, in this case, there's 2 parts: the verisign part, and the godaddy part. By 'parent parser' do you mean another part-parser, or parsers/base?

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

No branches or pull requests

2 participants