@nictool/dns-resource-record 1.2.0 → 1.2.2

package/.codeclimate.yml

@@ -0,0 +1,25 @@
+checks:
+  return-statements:
+    enabled: false
+  similar-code:
+    enabled: false
+  file-lines:
+    config:
+      threshold: 500
+  method-lines:
+    config:
+      threshold: 50
+  method-complexity:
+    config:
+      threshold: 10
+
+plugins:
+  eslint:
+    enabled: true
+    channel: 'eslint-8'
+    config:
+      config: '.eslintrc.yaml'
+
+ratings:
+  paths:
+    - '**.js'

package/.github/FUNDING.yml

@@ -0,0 +1,3 @@
+# These are supported funding model platforms
+
+github: msimerson

package/.github/workflows/ci.yml

@@ -0,0 +1,43 @@
+name: CI
+
+on:
+  push:
+  pull_request:
+
+env:
+  CI: true
+
+jobs:
+  lint:
+    uses: NicTool/.github/.github/workflows/lint.yml@main
+
+  coverage:
+    uses: NicTool/.github/.github/workflows/coverage.yml@main
+    secrets: inherit
+
+  test:
+    needs: get-lts
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
+        node-version: ${{ fromJson(needs.get-lts.outputs.active) }}
+      fail-fast: false
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        name: Node ${{ matrix.node-version }} on ${{ matrix.os }}
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: npm install
+      - run: npm test
+
+  get-lts:
+    needs: lint
+    runs-on: ubuntu-latest
+    steps:
+      - id: get
+        uses: msimerson/node-lts-versions@v1
+    outputs:
+      lts: ${{ steps.get.outputs.lts }}
+      active: ${{ steps.get.outputs.active }}

package/.github/workflows/codeql.yml

@@ -0,0 +1,14 @@
+name: CodeQL
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    # The branches below must be a subset of the branches above
+    branches: [main]
+  schedule:
+    - cron: '18 7 * * 4'
+
+jobs:
+  codeql:
+    uses: NicTool/.github/.github/workflows/codeql.yml@main

package/.github/workflows/publish.yml

@@ -0,0 +1,16 @@
+name: publish
+
+on:
+  push:
+    branches:
+      - main
+    paths:
+      - package.json
+
+env:
+  CI: true
+
+jobs:
+  publish:
+    uses: NicTool/.github/.github/workflows/publish.yml@main
+    secrets: inherit

package/.gitmodules

@@ -0,0 +1,3 @@
+[submodule ".release"]
+	path = .release
+	url = https://github.com/msimerson/.release

package/CHANGELOG.md

@@ -4,6 +4,20 @@ Notable changes to this project are documented in this file.
 
 #### Unreleased
 
+### [1.2.2] - 2024-11-17
+
+- dep(eslint): update to v9 & config
+- chore: tighten up fromBind regex
+- packaging tweaks (#43)
+
+### [1.2.1] - 2024-03-10
+
+- fix(nsec3param): fixed setHash fname typo
+- feat(SVCB,HTTPS): add record support (#29)
+- feat(OPENGPGPKEY): improved bindline parser, added test
+- feat(throwHelp): far more useful error messages (#41)
+- feat(A,MX,AAAA,CAA,IPSECKEY): added getCanonical
+
 ### [1.2.0] - 2024-03-07
 
 - feat(index): export typeMap (lookup table for type to id)
@@ -286,3 +300,7 @@ Notable changes to this project are documented in this file.
 [1.1.5]: https://github.com/NicTool/dns-resource-record/releases/tag/1.1.5
 [1.1.6]: https://github.com/NicTool/dns-resource-record/releases/tag/1.1.6
 [1.1.8]: https://github.com/NicTool/dns-resource-record/releases/tag/1.1.8
+[1.2.1]: https://github.com/NicTool/dns-resource-record/releases/tag/1.2.1
+[1.2.2]: https://github.com/NicTool/dns-resource-record/releases/tag/v1.2.2
+[1.1.7]: https://github.com/NicTool/dns-resource-record/releases/tag/1.1.7
+[1.2.0]: https://github.com/NicTool/dns-resource-record/releases/tag/1.2.0

package/DEVELOP.md

@@ -0,0 +1,9 @@
+# Release process
+
+In your local repo:
+
+```sh
+git submodule update --init --recursive
+```
+
+Read and follow the instructions in .release/README.md

package/README.md

@@ -10,7 +10,7 @@ DNS resource record parser, validator, importer, and exporter.
 This module is used to:
 
 - validate well formedness and RFC compliance of DNS resource records
-- import RRs from and from the following formats:
+- import RRs to and from the following formats:
 
 |                      **RR format**                       |     **import**     |     **export**     |
 | :------------------------------------------------------: | :----------------: | :----------------: |
@@ -20,7 +20,7 @@ This module is used to:
 |                       **MaraDNS**                        |                    | :white_check_mark: |
 |                          **JS**                          | :white_check_mark: | :white_check_mark: |
 
-This package intends to import and export RFC compliant DNS resource records. Please [raise an issue](https://github.com/NicTool/dns-resource-record/issues) if you a valid resource record fails to pass or an invalid resource record passes.
+This package intends to import and export RFC compliant DNS resource records. Please [raise an issue](https://github.com/NicTool/dns-resource-record/issues) if a valid resource record fails to pass or an invalid resource record passes.
 
 This package is for working with _individual_ Resource Records. For working with zones of RRs, use [dns-zone](https://github.com/NicTool/dns-zone).
 
@@ -196,6 +196,7 @@ PRs are welcome, especially PRs with tests.
 |   **DNSKEY**   | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 |     **DS**     | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 |   **HINFO**    | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
+|   **HTTPS**    |                    |                    |                    |                    |
 |  **IPSECKEY**  | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 |    **KEY**     |                    |                    |                    |                    |
 |    **LOC**     | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
@@ -215,6 +216,7 @@ PRs are welcome, especially PRs with tests.
 |    **SPF**     | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 |    **SRV**     | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 |   **SSHFP**    | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
+|    **SVCB**    |                    |                    |                    |                    |
 |    **TLSA**    | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 |    **TSIG**    |                    |                    |                    |                    |
 |    **TXT**     | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
@@ -237,6 +239,8 @@ PRs are welcome, especially PRs with tests.
 
 - [Dictionary of DNS terms](https://nictool.github.io/web/Dictionary)
 - [Wikipedia, List of DNS Record Types](https://en.wikipedia.org/wiki/List_of_DNS_record_types)
+- @nictool/[dns-zone](https://www.npmjs.com/package/@nictool/dns-zone)
+- @nictool/[dns-nameserver](https://www.npmjs.com/package/@nictool/dns-nameserver)
 
 ## TODO
 

package/eslint.config.mjs

@@ -0,0 +1,41 @@
+import globals from 'globals'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+import js from '@eslint/js'
+import { FlatCompat } from '@eslint/eslintrc'
+
+const __filename = fileURLToPath(import.meta.url)
+const __dirname = path.dirname(__filename)
+const compat = new FlatCompat({
+  baseDirectory: __dirname,
+  recommendedConfig: js.configs.recommended,
+  allConfig: js.configs.all,
+})
+
+export default [
+  {
+    languageOptions: {
+      ecmaVersion: 'latest',
+      globals: {
+        ...globals.node,
+        ...globals.browser,
+        ...globals.mocha,
+      },
+      sourceType: 'module',
+    },
+
+    rules: {
+      // 'no-undef': [ 'warn' ],
+      'no-unused-vars': [
+        'error',
+        {
+          args: 'none',
+        },
+      ],
+
+      'dot-notation': 'error',
+      'prefer-const': 'warn',
+    },
+  },
+  js.configs.recommended,
+]

package/index.js

@@ -10,6 +10,7 @@ import DNAME from './rr/dname.js'
 import DNSKEY from './rr/dnskey.js'
 import DS from './rr/ds.js'
 import HINFO from './rr/hinfo.js'
+import HTTPS from './rr/https.js'
 import IPSECKEY from './rr/ipseckey.js'
 import KEY from './rr/key.js'
 import LOC from './rr/loc.js'
@@ -29,6 +30,7 @@ import SOA from './rr/soa.js'
 import SPF from './rr/spf.js'
 import SRV from './rr/srv.js'
 import SSHFP from './rr/sshfp.js'
+import SVCB from './rr/svcb.js'
 import TLSA from './rr/tlsa.js'
 import TSIG from './rr/tsig.js'
 import TXT from './rr/txt.js'
@@ -47,6 +49,7 @@ export {
   DNSKEY,
   DS,
   HINFO,
+  HTTPS,
   IPSECKEY,
   KEY,
   LOC,
@@ -66,6 +69,7 @@ export {
   SOA,
   SPF,
   SRV,
+  SVCB,
   TLSA,
   TSIG,
   TXT,
@@ -84,6 +88,7 @@ for (const c of [
   DNSKEY,
   DS,
   HINFO,
+  HTTPS,
   IPSECKEY,
   KEY,
   LOC,
@@ -103,6 +108,7 @@ for (const c of [
   SOA,
   SPF,
   SRV,
+  SVCB,
   TLSA,
   TSIG,
   TXT,

package/package.json

@@ -1,13 +1,19 @@
 {
   "name": "@nictool/dns-resource-record",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "description": "DNS Resource Records",
   "main": "index.js",
+  "file": [
+    "lib",
+    "rr",
+    "CHANGELOG.md",
+    "rr.js"
+  ],
   "type": "module",
   "scripts": {
     "format:check": "npm run prettier; npm run lint",
     "format": "npm run prettier -- --write && npm run lint --fix",
-    "lint": "npx eslint **/*.js",
+    "lint": "npx eslint@9 **/*.js",
     "lint:fix": "npm run lint -- --fix",
     "prettier": "npx prettier --ignore-path .gitignore --check .",
     "prettier:fix": "npx prettier --ignore-path .gitignore --write .",
@@ -39,6 +45,9 @@
   },
   "homepage": "https://github.com/NicTool/dns-resource-record#readme",
   "devDependencies": {
-    "mocha": "^10.3.0"
+    "@eslint/js": "^9.15.0",
+    "eslint": "^9.15.0",
+    "globals": "^15.12.0",
+    "mocha": "^10.8.2"
   }
 }

package/rr/a.js

@@ -1,4 +1,4 @@
-import net from 'net'
+import net from 'node:net'
 
 import RR from '../rr.js'
 
@@ -9,8 +9,8 @@ export default class A extends RR {
 
   /****** Resource record specific setters   *******/
   setAddress(val) {
-    if (!val) throw new Error('A: address is required')
-    if (!net.isIPv4(val)) throw new Error('A address must be IPv4')
+    if (!val) this.throwHelp('A: address is required')
+    if (!net.isIPv4(val)) this.throwHelp('A address must be IPv4')
     this.set('address', val)
   }
 
@@ -30,6 +30,16 @@ export default class A extends RR {
     return 1
   }
 
+  getCanonical() {
+    return {
+      owner: 'host.example.com.',
+      class: 'IN',
+      ttl: 3600,
+      type: 'A',
+      address: '192.0.2.127',
+    }
+  }
+
   /******  IMPORTERS   *******/
   fromTinydns(opts) {
     // +fqdn:ip:ttl:timestamp:lo

package/rr/aaaa.js

@@ -1,4 +1,4 @@
-import net from 'net'
+import net from 'node:net'
 
 import RR from '../rr.js'
 import * as TINYDNS from '../lib/tinydns.js'
@@ -10,8 +10,8 @@ export default class AAAA extends RR {
 
   /****** Resource record specific setters   *******/
   setAddress(val) {
-    if (!val) throw new Error('AAAA: address is required')
-    if (!net.isIPv6(val)) throw new Error(`AAAA: address must be IPv6 (${val})`)
+    if (!val) this.throwHelp('AAAA: address is required')
+    if (!net.isIPv6(val)) this.throwHelp(`AAAA: address must be IPv6 (${val})`)
 
     this.set('address', this.expand(val.toLowerCase())) // lower case: RFC 5952
   }
@@ -36,6 +36,16 @@ export default class AAAA extends RR {
     return 28
   }
 
+  getCanonical() {
+    return {
+      owner: 'host.example.com.',
+      address: '2001:0db8:0020:000a:0000:0000:0000:0004',
+      class: 'IN',
+      ttl: 3600,
+      type: 'A',
+    }
+  }
+
   /******  IMPORTERS   *******/
   fromTinydns(opts) {
     const str = opts.tinyline
@@ -45,7 +55,7 @@ export default class AAAA extends RR {
       case ':':
         // GENERIC  =>  :fqdn:28:rdata:ttl:timestamp:lo
         ;[fqdn, n, rdata, ttl, ts, loc] = str.substring(1).split(':')
-        if (n != 28) throw new Error('AAAA fromTinydns, invalid n')
+        if (n != 28) this.throwHelp('AAAA fromTinydns, invalid n')
         ip = TINYDNS.octalToHex(rdata)
           .match(/([0-9a-fA-F]{4})/g)
           .join(':')

package/rr/caa.js

@@ -11,7 +11,7 @@ export default class CAA extends RR {
     this.is8bitInt('CAA', 'flags', val)
 
     if (![0, 128].includes(val)) {
-      throw new Error(`CAA flags ${val} not recognized, ${this.citeRFC()}`)
+      this.throwHelp(`CAA flags ${val} not recognized`)
     }
 
     this.set('flags', val)
@@ -19,12 +19,12 @@ export default class CAA extends RR {
 
   setTag(val) {
     if (typeof val !== 'string' || val.length < 1 || /[^a-z0-9]/.test(val))
-      throw new Error(
-        `CAA tag must be a sequence of ASCII letters and numbers in lowercase, ${this.citeRFC()}`,
+      this.throwHelp(
+        `CAA tag must be a sequence of ASCII letters and numbers in lowercase`,
       )
 
     if (!['issue', 'issuewild', 'iodef'].includes(val)) {
-      throw new Error(`CAA tag ${val} not recognized: ${this.citeRFC()}`)
+      this.throwHelp(`CAA tag ${val} not recognized`)
     }
     this.set('tag', val)
   }
@@ -35,16 +35,14 @@ export default class CAA extends RR {
     if (this.isQuoted(val)) {
       val = val.replace(/^["']|["']$/g, '') // strip quotes
     } else {
-      // if (/\s/.test(val)) throw new Error(`CAA value may not have spaces unless quoted: RFC 8659`)
+      // if (/\s/.test(val)) this.throwHelp(`CAA value may not have spaces unless quoted`)
     }
 
     // check if val starts with one of iodefSchemes
     if (this.get('tag') === 'iodef') {
       const iodefSchemes = ['mailto:', 'http:', 'https:']
       if (!iodefSchemes.filter((s) => val.startsWith(s)).length) {
-        throw new Error(
-          `CAA value must have valid iodefScheme prefix, ${this.citeRFC()}`,
-        )
+        this.throwHelp(`CAA value must have valid iodefScheme prefix`)
       }
     }
 
@@ -71,11 +69,23 @@ export default class CAA extends RR {
     return 257
   }
 
+  getCanonical() {
+    return {
+      owner: 'example.com.',
+      ttl: 3600,
+      class: 'IN',
+      type: 'CAA',
+      flags: 0,
+      tag: 'issue',
+      value: 'http://letsencrypt.org',
+    }
+  }
+
   /******  IMPORTERS   *******/
   fromTinydns(opts) {
     // CAA via generic, :fqdn:n:rdata:ttl:timestamp:lo
     const [fqdn, n, rdata, ttl, ts, loc] = opts.tinyline.substring(1).split(':')
-    if (n != 257) throw new Error('CAA fromTinydns, invalid n')
+    if (n != 257) this.throwHelp('CAA fromTinydns, invalid n')
 
     const flags = TINYDNS.octalToUInt8(rdata.substring(0, 4))
     const taglen = TINYDNS.octalToUInt8(rdata.substring(4, 8))
@@ -98,10 +108,12 @@ export default class CAA extends RR {
 
   fromBind(opts) {
     // test.example.com  3600  IN  CAA flags, tags, value
-    const fields = opts.bindline.match(
-      /^([^\s]+)\s+([0-9]+)\s+(\w+)\s+(\w+)\s+([0-9]+)\s+(\w+)\s+("[^"]+"|[^\s]+?)\s*$/i,
-    )
-    if (!fields) throw new Error(`unable to parse: ${opts.bindline}`)
+    const fields = opts.bindline
+      .trim()
+      .match(
+        /^([\S]+)\s+([0-9]{1,10})\s+(IN)\s+(CAA)\s+([0-9]+)\s+(\w+)\s+("[^"]+"|[\S]+?)$/i,
+      )
+    if (!fields) this.throwHelp(`unable to parse: ${opts.bindline}`)
 
     const [owner, ttl, c, type, flags, tag, value] = fields.slice(1)
     return new CAA({

package/rr/cname.js

@@ -1,4 +1,4 @@
-import net from 'net'
+import net from 'node:net'
 
 import RR from '../rr.js'
 
@@ -12,10 +12,10 @@ export default class CNAME extends RR {
     // A <domain-name> which specifies the canonical or primary
     // name for the owner.  The owner name is an alias.
 
-    if (!val) throw new Error('CNAME: cname is required')
+    if (!val) this.throwHelp('CNAME: cname is required')
 
     if (net.isIPv4(val) || net.isIPv6(val))
-      throw new Error(`CNAME: cname must be a FQDN: RFC 2181`)
+      this.throwHelp(`CNAME: cname must be a FQDN: RFC 2181`)
 
     if (!this.isFullyQualified('CNAME', 'cname', val)) return
     if (!this.isValidHostname('CNAME', 'cname', val)) return

package/rr/dname.js

@@ -1,4 +1,4 @@
-import net from 'net'
+import net from 'node:net'
 
 import RR from '../rr.js'
 import * as TINYDNS from '../lib/tinydns.js'
@@ -10,10 +10,10 @@ export default class DNAME extends RR {
 
   /****** Resource record specific setters   *******/
   setTarget(val) {
-    if (!val) throw new Error('DNAME: target is required')
+    if (!val) this.throwHelp('DNAME: target is required')
 
     if (net.isIPv4(val) || net.isIPv6(val))
-      throw new Error(`DNAME: target must be a domain name, ${this.citeRFC()}`)
+      this.throwHelp(`DNAME: target must be a domain name`)
 
     this.isFullyQualified('DNAME', 'target', val)
     this.isValidHostname('DNAME', 'target', val)
@@ -42,7 +42,7 @@ export default class DNAME extends RR {
   fromTinydns(opts) {
     // DNAME via generic, :fqdn:n:rdata:ttl:timestamp:lo
     const [fqdn, n, rdata, ttl, ts, loc] = opts.tinyline.substring(1).split(':')
-    if (n != 39) throw new Error('DNAME fromTinydns, invalid n')
+    if (n != 39) this.throwHelp('DNAME fromTinydns, invalid n')
 
     return new DNAME({
       type: 'DNAME',

package/rr/dnskey.js

@@ -13,8 +13,7 @@ export default class DNSKEY extends RR {
     this.is16bitInt('DNSKEY', 'flags', val)
 
     // the possible values are: 0, 256, and 257; RFC 4034
-    if (![0, 256, 257].includes(val))
-      throw new Error(`DNSKEY: flags invalid, ${this.citeRFC()}`)
+    if (![0, 256, 257].includes(val)) this.throwHelp(`DNSKEY: flags invalid`)
 
     this.set('flags', val)
   }
@@ -24,8 +23,7 @@ export default class DNSKEY extends RR {
     this.is8bitInt('DNSKEY', 'protocol', val)
 
     // The Protocol Field MUST be represented as an unsigned decimal integer with a value of 3.
-    if (![3].includes(val))
-      throw new Error(`DNSKEY: protocol invalid, ${this.citeRFC()}`)
+    if (![3].includes(val)) this.throwHelp(`DNSKEY: protocol invalid`)
 
     this.set('protocol', val)
   }
@@ -37,16 +35,13 @@ export default class DNSKEY extends RR {
     // https://www.iana.org/assignments/dns-sec-alg-numbers/dns-sec-alg-numbers.xhtml
     // 1=RSA/MD5, 2=DH, 3=DSA/SHA-1, 4=EC, 5=RSA/SHA-1
     if (![...Array(16).keys(), 253, 254].includes(val))
-      console.error(
-        `DNSKEY: algorithm (${val}) not recognized, ${this.citeRFC()}`,
-      )
+      console.error(`DNSKEY: algorithm (${val}) not recognized`)
 
     this.set('algorithm', val)
   }
 
   setPublickey(val) {
-    if (!val)
-      throw new Error(`DNSKEY: publickey is required, ${this.citeRFC()}`)
+    if (!val) this.throwHelp(`DNSKEY: publickey is required`)
 
     this.set('publickey', val)
   }
@@ -74,7 +69,7 @@ export default class DNSKEY extends RR {
     const match = opts.bindline.match(
       /^([^\s]+)\s+([0-9]+)\s+(\w+)\s+(\w+)\s+([0-9]+)\s+([0-9]+)\s+([0-9]+)\s+\s*(.*?)\s*$/,
     )
-    if (!match) throw new Error(`unable to parse DNSKEY: ${opts.bindline}`)
+    if (!match) this.throwHelp(`unable to parse DNSKEY: ${opts.bindline}`)
     const [owner, ttl, c, type, flags, protocol, algorithm, publickey] =
       match.slice(1)
 
@@ -92,7 +87,7 @@ export default class DNSKEY extends RR {
 
   fromTinydns(opts) {
     const [fqdn, n, rdata, ttl, ts, loc] = opts.tinyline.substring(1).split(':')
-    if (n != 48) throw new Error('DNSKEY fromTinydns, invalid n')
+    if (n != 48) this.throwHelp('DNSKEY fromTinydns, invalid n')
 
     const bytes = Buffer.from(TINYDNS.octalToChar(rdata), 'binary')
 

package/rr/ds.js

@@ -10,9 +10,8 @@ export default class DS extends RR {
   /****** Resource record specific setters   *******/
   setKeyTag(val) {
     // a 2 octet Key Tag field...in network byte order
-    if (!val) throw new Error(`DS: key tag is required`)
-    if (val.length > 2)
-      throw new Error(`DS: key tag is too long, ${this.citeRFC()}`)
+    if (!val) this.throwHelp(`DS: key tag is required`)
+    if (val.length > 2) this.throwHelp(`DS: key tag is too long`)
 
     this.set('key tag', val)
   }
@@ -20,20 +19,19 @@ export default class DS extends RR {
   setAlgorithm(val) {
     // 1=RSA/MD5, 2=DH, 3=DSA/SHA-1, 4=EC, 5=RSA/SHA-1
     if (![1, 2, 3, 4, 5, 253, 254].includes(val))
-      throw new Error(`DS: algorithm invalid, ${this.citeRFC()}`)
+      this.throwHelp(`DS: algorithm invalid`)
 
     this.set('algorithm', val)
   }
 
   setDigestType(val) {
-    if (![1, 2].includes(val))
-      throw new Error(`DS: digest type invalid, ${this.citeRFC()}`)
+    if (![1, 2].includes(val)) this.throwHelp(`DS: digest type invalid`)
 
     this.set('digest type', val)
   }
 
   setDigest(val) {
-    if (!val) throw new Error(`DS: digest is required, ${this.citeRFC()}`)
+    if (!val) this.throwHelp(`DS: digest is required`)
 
     this.set('digest', val)
   }
@@ -74,7 +72,7 @@ export default class DS extends RR {
 
   fromTinydns(opts) {
     const [fqdn, n, rdata, ttl, ts, loc] = opts.tinyline.substring(1).split(':')
-    if (n != 43) throw new Error('DS fromTinydns, invalid n')
+    if (n != 43) this.throwHelp('DS fromTinydns, invalid n')
 
     const binRdata = Buffer.from(TINYDNS.octalToChar(rdata), 'binary')
 

package/rr/hinfo.js

@@ -9,12 +9,12 @@ export default class HINFO extends RR {
 
   /****** Resource record specific setters   *******/
   setCpu(val) {
-    if (val.length > 255) throw new Error('HINFO cpu cannot exceed 255 chars')
+    if (val.length > 255) this.throwHelp('HINFO cpu cannot exceed 255 chars')
     this.set('cpu', val.replace(/^["']|["']$/g, ''))
   }
 
   setOs(val) {
-    if (val.length > 255) throw new Error('HINFO os cannot exceed 255 chars')
+    if (val.length > 255) this.throwHelp('HINFO os cannot exceed 255 chars')
     this.set('os', val.replace(/^["']|["']$/g, ''))
   }
 
@@ -41,10 +41,12 @@ export default class HINFO extends RR {
   /******  IMPORTERS   *******/
   fromBind(opts) {
     // test.example.com  3600  IN  HINFO   DEC-2060 TOPS20
-    const match = opts.bindline.match(
-      /([^\s]+)\s+([0-9]+)\s+(IN)\s+(HINFO)\s+("[^"]+"|[^\s]+)\s+("[^"]+"|[^\s]+)/i,
-    )
-    if (!match) throw new Error(`unable to parse HINFO: ${opts.bindline}`)
+    const match = opts.bindline
+      .trim()
+      .match(
+        /^([\S]+)\s+([0-9]{1,10})\s+(IN)\s+(HINFO)\s+("[^"]+"|[\S]+)\s+("[^"]+"|[\S]+)/i,
+      )
+    if (!match) this.throwHelp(`unable to parse HINFO: ${opts.bindline}`)
     const [owner, ttl, c, type, cpu, os] = match.slice(1)
 
     return new HINFO({