Skip to content

Fix cases where dig -r wasn't working#1972

Merged
drwetter merged 2 commits into3.1devfrom
fix_dig-r
Sep 1, 2021
Merged

Fix cases where dig -r wasn't working#1972
drwetter merged 2 commits into3.1devfrom
fix_dig-r

Conversation

@drwetter
Copy link
Copy Markdown
Collaborator

@drwetter drwetter commented Sep 1, 2021

  • the ignore ~/.digrc option from dig is now parsed from the builtin help
  • there was a potential DNS call which is now avoided
  • the +noidnout check is now parsed from the builtin help
  • the OPENSSL_CONF="" in check_resolver_bins() was moved a few lines higher to avoid other errors in the terminal
  • redo check for noidnout by also using the help function of dig

Tested (also) on (EOL) Ubuntu 14.04 which only has dig in an older version

See also #1950

* the ignore ~/.digrc option from dig is now parsed from the builtin help
* there was a potential DNS call which is now avoided
* for +noidnout check however there's a call to invalid. added
* the OPENSSL_CONF="" in check_resolver_bins() was moved a few lines
  higher to avoid other errors in the terminal

Tested on (EOL) Ubuntu 14.04 which only has dig in an older version

See also #1950
* add -q in grep statement to make sure the console stays clean
* redo check for noidnout by also using the help function of dig
drwetter added a commit that referenced this pull request Sep 1, 2021
Similar to #1972.

* the ignore ~/.digrc option from dig is now parsed from the builtin help
* there was a potential DNS call which is now avoided
* the +noidnout check is now parsed from the builtin help
* the OPENSSL_CONF="" in check_resolver_bins() was moved a few lines higher to avoid other errors in the terminal
* redo check for noidnout by also using the help function of dig

See also #1949
@drwetter drwetter merged commit 05ef9f9 into 3.1dev Sep 1, 2021
@drwetter drwetter deleted the fix_dig-r branch September 28, 2021 09:47
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

Successfully merging this pull request may close these issues.

1 participant