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

[auth] [janitor] Mop up some Lua records code #15235

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

miodvallat
Copy link
Contributor

Short description

Today is a new moon, so it's perfect time to work on Lua support in PowerDNS.

This purely janitorial PR splits the monster function setupLuaRecords into pieces, so that every single Lua helper function (such as createReverse, ifurlup...) can be found in its own function.

This will help working in this area in the future.

Probably better reviewed in a per-commit basis, with whitespace changes ignored in the first commit.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

This does not change anything, but reduces indent and makes the code
easier to read. No functional change yet.
@miodvallat miodvallat added auth janitor Janitor-style cleaning tasks which do not change functionality labels Feb 28, 2025
@coveralls
Copy link

coveralls commented Feb 28, 2025

Pull Request Test Coverage Report for Build 13586761189

Details

  • 0 of 559 (0.0%) changed or added relevant lines in 1 file are covered.
  • 11505 unchanged lines in 206 files lost coverage.
  • Overall coverage increased (+3.4%) to 64.465%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pdns/lua-record.cc 0 559 0.0%
Files with Coverage Reduction New Missed Lines %
ext/lmdb-safe/lmdb-typed.cc 1 89.66%
pdns/auth-catalogzone.hh 1 66.67%
pdns/dnsdistdist/dnsdist-kvs.hh 1 55.0%
pdns/dnsdistdist/dnsdist-lua-hooks.cc 1 94.59%
pdns/dnsdistdist/dnsdist-session-cache.cc 1 62.86%
pdns/dnswriter.hh 1 76.6%
pdns/dynlistener.hh 1 0.0%
pdns/inflighter.cc 1 90.48%
pdns/qtype.hh 1 86.96%
pdns/recursordist/nod.hh 1 92.59%
Totals Coverage Status
Change from base Build 13576694012: 3.4%
Covered Lines: 127628
Relevant Lines: 167023

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auth janitor Janitor-style cleaning tasks which do not change functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants