Skip to content
Peter van Dijk edited this page May 5, 2021 · 46 revisions

With reference to https://en.wiktionary.org/wiki/yak_shaving, sometimes somebody will run into something that 'could be nicer' but does not warrant an issue ticket. Instead, we note those things here. Pull requests fixing these things are welcomed! Feel free to add to this list.

The yaks are numbered so a PR can easily reference them.. Please use ~bla bla~ to strike out finished items, adding 'PR #1234after the closing~`.

  • (auth at least) ./configure --help has alignment issues PR #7005
  • (all products) 15:10Z <lieter> another yak I have with confige is our inconsistent use of --enable-X and --with-X 15:11Z <lieter> --enable are _our_ programs's options and --with-X should be external deps PR #7026
  • (auth at least) README has instructions for Debian 8.0, should not say '.0' and should be updated for 9 PR #7027
  • recursor and dnsdist rely on symlinks into the 'main' pdns source tree. Travis should check for dangling symlinks. PR #7028
  • figure out Travis/python/SSL warnings and do something about them PR #7106
  • generally improve quality of Travis logging output
  • in 49f72da134c234603abc5d270f73ea6191dceeff, we imported timegm from dietlibc, presumably(!) because Solaris does not have it. Figure out if that is still relevant today, otherwise, get rid of it.
  • in https://dnsdist.org/reference/config.html?highlight=setup, there is a visible :ref: (to "Healthcheck") that should be a link
  • document that dnsdist newServer() returns the server object
  • parallelize Travis builds or at least check if this is still applicable (also see here)
  • check whether _blank is an issue in our code (targetblank)
  • LGTM alerts
  • Make sure LGTM only builds the right products on the right branches (i.e. no auth builds on a rec branch)
  • We use unsigned int and other int types without size identifiers. It would be nice to switch to properly typed integers (like uint32_t).
  • Remove all references to GOST (some already done, some in #7187, more to do)
  • pdns-4.1.5.tar.gz accidentally contains regression-tests/zones/unit.test, perhaps other undesired files as well. Figure out a way to notice this (a list in git?)
  • some of the Lua examples in the docs, and example *.lua files in the tree, use print(), they should use pdnslog
  • fix this style nit: https://github.com/PowerDNS/pdns/pull/6531#issuecomment-384242153
  • pdns/docs/http-api/tsigkey.rst:63: WARNING: Definition list ends without a blank line; unexpected unindent.
  • decide if we want to do anything (document?) about the universe thing in https://mailman.powerdns.com/pipermail/pdns-users/2018-December/025685.html
  • rec_control --version
  • Flake8 testing
  • ifurlup (Python function, in Preset variables) it's a Lua function
  • https://doc.powerdns.com/authoritative/http-api/cryptokey.html#cryptokey - the underlying swagger yaml says keytype is an enum, but the docs do not reflect that
  • https://doc.powerdns.com/authoritative/backends/oracle.html - the Warning is in the middle of a sentence
  • sdig does not report parser errors. For example, a reply packet (in hex here) 9a0e83830001000000010001023432033235330331353802393807696e2d61646472046172706100000c0001c0170006000100002a1a002a017a046172696e036e65740007646e732d6f7073c1f6783962e60000070800000384000a8c0000002a300000291000000000000000 has a forward label compression reference that is caught by dnsname.cc:130 (-> 130 throw std::range_error("Found a forward reference during label decompression");) but sdig throws that exception away, somehow
  • checking for library containing mdb_env_open... no does not abort auth ./configure
  • recursor --enable-unit-tests --enable-ubsan: testrunner fails to link (undefined reference to ``typeinfo for RecursorLua4') unless you add SANITIZER_FLAGS='-fno-sanitize=vptr'