Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • BIND BIND
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 632
    • Issues 632
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 90
    • Merge requests 90
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ISC Open Source ProjectsISC Open Source Projects
  • BINDBIND
  • Merge requests
  • !548

QNAME minimization remake

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Witold Krecicki requested to merge 16-qname-minimization-newfctx into master Jul 23, 2018
  • Overview 10
  • Commits 15
  • Pipelines 40
  • Changes 30

Instead of doing everything in one fetch context we create separate fctxs for each minimized query.

Also added is loop detection in queries, which happens more often in qname minimization, classic example is:

example1. IN NS ns.example2. example2. IN NS ns.example1.

In a hot cache situation named would timeout on this without even sending any queries, which IMHO is an odd behaviour. Since some of the tests (e.g. reclimit) directly depended on this behaviour they are now broken.

Edited Oct 23, 2018 by Witold Krecicki
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: 16-qname-minimization-newfctx