Skip to content

[dns] - resolveTxt ttl flag #39528

Closed
Closed
[dns] - resolveTxt ttl flag#39528
Labels
caresIssues and PRs related to the c-ares dependency or the cares_wrap binding.dnsIssues and PRs related to the dns subsystem.duplicateIssues and PRs that are duplicates of other issues or PRs.feature requestIssues requesting new Node.js features.

Description

@tanguyantoine
tanguyantoine
opened on Jul 26, 2021

Is your feature request related to a problem? Please describe.
I would like to implement some application cache on top of dns.resolveTxt but TTL are not parsed nor returned by the dns lib

Describe the solution you'd like
The same way as https://nodejs.org/docs/latest-v15.x/api/dns.html#dns_dns_resolve4_hostname_options_callback I would love to have the ability to provide a ttl boolean flag so that I retrieved the dnx txt records with their ttl

Describe alternatives you've considered
None

Note: I updated unit tests to show case how I would like to use this API: #39529

Activity

  1. gireeshpunathil commented on Aug 7, 2021

    @gireeshpunathil
    gireeshpunathil
    on Aug 7, 2021
    Member

    @nodejs/dns

  2. tniessen commented on Aug 8, 2021

    @tniessen
    tniessen
    on Aug 8, 2021
    Member

    For reference, the ttl option was discussed in #5893 and added in #9296.

    The implementation appears to achieve this using ares_parse_a_reply and ares_parse_aaaa_reply, respectively. However, other parse functions, such as ares_parse_ns_reply, ares_parse_ptr_reply, ares_parse_mx_reply, ares_parse_txt_reply_ext, etc. do not appear to support retrieving the TTL of the records.

  3. tniessen
    changed the title [-][dsn] - resolveTxt ttl flag[/-] [+][dns] - resolveTxt ttl flag[/+] on Aug 8, 2021
  4. tniessen
    added
    dnsIssues and PRs related to the dns subsystem.
    feature requestIssues requesting new Node.js features.
    caresIssues and PRs related to the c-ares dependency or the cares_wrap binding.
    on Aug 8, 2021

  5. tniessen commented on Aug 8, 2021

    @tniessen
    tniessen
    on Aug 8, 2021
    Member

    This appears to be a duplicate of #39399. Feel free to reopen if that's not correct.

  6. tniessen
    closed this as completedon Aug 8, 2021
  7. tniessen
    added
    duplicateIssues and PRs that are duplicates of other issues or PRs.
    on Aug 8, 2021

  8. tanguyantoine commented on Aug 10, 2021

    @tanguyantoine
    tanguyantoine
    on Aug 10, 2021
    Author

    haha I'm late ^^ 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    caresIssues and PRs related to the c-ares dependency or the cares_wrap binding.dnsIssues and PRs related to the dns subsystem.duplicateIssues and PRs that are duplicates of other issues or PRs.feature requestIssues requesting new Node.js features.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions