Adding 5 more parsers for Whois servers #175

Closed
wants to merge 1 commit into from

2 participants

@SophosLabs

The following parsers were added:

whois.discount-domain.com, whois.domain.com, whois.register.com, whois.register.it, whois.rrpproxy.net.

SophosLabs Add new parsers for the following whois servers: discount-domain.com,…
… domain.com, register.com,

register.it, rrpproxy.net
e0d9834
@weppos
Owner

Hi @SophosLabs, thanks for your patch. I would really love to merge the changes, however the patch requires a bit of cleanup before being able to merge it.

First of all, the patch is missing the necessary tests. I cannot merge a change without tests.
Also, it would be perfect if you can split the patch in 5 different pull requests, one for each parser. In this way we can review them separately without causing overlap in discussions.

-- Simone

@weppos weppos closed this Jun 28, 2012
@weppos weppos was assigned Jun 28, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment