Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dnsdist: Implement cache expungeByName support in the APIs #6154

Closed
atosatto opened this issue Jan 7, 2018 · 2 comments · Fixed by #12473
Closed

dnsdist: Implement cache expungeByName support in the APIs #6154

atosatto opened this issue Jan 7, 2018 · 2 comments · Fixed by #12473

Comments

@atosatto
Copy link
Contributor

atosatto commented Jan 7, 2018

  • Program: dnsdist
  • Issue type: Feature request

Short description

Currently, the dnsdist APIs do not expose any endpoint to manipulate the records cache (e.g. evict some records).
This issue aims to request the evaluation of the implementation an endpoint to manipulate via APIs dnsdist's per-pools records cache.

Usecase & Description

Deploying dnsdist as additional cache/protection layer in front of Authoritative DNS servers require specifically maintenance operations in order to make sure to evict dnsdist records cache everytime a DNS record is updated on the backend Authoritative DNS servers.

Cache eviction operations are right now are only available via TCP control socket

dnsdist -e 'getPool(""):getCache():expungeByName(newDNSName("www.example.net"))'

Exposing the same functionality over HTTP RESTful APIs would ease the life-cycle management (via custom HTTP web-hooks) of the records cache without having to artificially lower its maximum TTL configuration into the dnsdist.conf file

@pieterlexis
Copy link
Contributor

After a small discussion with @rgacogne, we decided that this endpoint should live in a new REST API (v2) that should be defined before working on this. We are not sure whether to use swagger-codegen or use the existing REST infra we have. But defining (parts of) the API needs to happen before implementation.

@dch
Copy link

dch commented Jun 4, 2021

I used the control socket previously, but knocked up Just Enough Lua today for this #10468

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

Successfully merging a pull request may close this issue.

5 participants