anbertoli / ariadne-dns Goto Github PK

The ariadne::nameserver::zones module should be completed with unit testing.

The ariadne::shared::dns module should be completed with unit tests.

I am looking at the file src/common/log.rs and noticed that it is largely similar in the calling semantics to the crate log, such as log::error!(), etc. I'm genuinely curious, is there a particular reason that you decided to write the logging functionality?

The codebase should have testing for several modules in addition to proper integration testing. This issue tracks several sub tasks related to testing.

• #1
• #3

Currently, we are using a thread pool to handle concurrency. We should migrate to ´async´ to give the server the ability to handle more concurrency.

The resolver cache must have a max size. When the cache is full, an algorithm (e.g LRU, FIFO etc.) must choose which items to discard to make room for the new ones.