whois-parser icon indicating copy to clipboard operation
whois-parser copied to clipboard

parsers: whois.cira.ca: Add new version support for 2019 whois output

Open xaf opened this issue 6 years ago • 4 comments

Most recent whois output from CIRA, with year 2019, has a very different output than what was expected in the past. This adds support for it while keeping support for previous versions of the whois output.

This requires the changes introduced in PR https://github.com/weppos/whois-parser/pull/130 to work properly (as issue with the missing new line otherwise).

Fixes #126

xaf avatar Nov 07 '19 20:11 xaf

Looking good! Fixes #126.

joallard avatar Feb 21 '20 21:02 joallard

I'm finding that this fix is needed for the .ca domains.

If we're not comfortable with the fix in #130, Is it possible to move the fix for the unterminated (with newline lines into the parser (lib/whois/parsers/whois.cira.ca.rb) with something like:

      def content_for_scanner
        super
        @content_for_scanner << "\n" unless @content_for_scanner.end_with?("\n")
        @content_for_scanner
      end

How can we move this PR forward?

ericgascoine avatar Jun 17 '20 21:06 ericgascoine

I just answered to @joallard on #130 to have this move forward. Who has merge rights on this repository though ?

xaf avatar Jul 14 '20 21:07 xaf

Thanks for this @XaF, for the record I ended up starting my fork in which I merged some of the pending PRs and provided more fixes. I didn't cherry-pick your PR here because it was keeping a lot of legacy code and adding on top, instead I refactored the parser to use the standard ICANN Compliant parser and updated all tests in ed7068c39a9821b084eb59416e49d3f0a7567d83. There's no reason to keep supporting the outdated version (unless you see one?), whois.cira.ca server is not going to return older versions any more. This version is working well for the couple hundreds of .ca domains I have. So people who need this can use my fork in the meantime:

gem 'whois-parser', github: 'jarthod/whois-parser' # Unofficial but more up-to-date fork, check status at https://github.com/jarthod/whois-parser

jarthod avatar Nov 25 '21 14:11 jarthod