zonemaster-engine icon indicating copy to clipboard operation
zonemaster-engine copied to clipboard

All requests are not properly cached

Open ghost opened this issue 4 years ago • 2 comments

It seems that all requests performed within the Nameserver's module are not properly stored in cache. Replaying these tests will result in network traffic.

Storing the data with the CLI:

zonemaster-cli --save zonemaster.net.data zonemaster.net

Using the cache for each module independently, it seems that the leak comes from the Nameserver's tests and more precisely from Nameserver/Nameserver03, Nameserver/Nameserver08 and Nameserver/Nameserver09.

zonemaster-cli --test nameserver --restore zonemaster.net.data zonemaster.net
zonemaster-cli --test nameserver/nameserver03 --restore zonemaster.net.data zonemaster.net
zonemaster-cli --test nameserver/nameserver08 --restore zonemaster.net.data zonemaster.net
zonemaster-cli --test nameserver/nameserver09 --restore zonemaster.net.data zonemaster.net

Addresses https://github.com/zonemaster/zonemaster-backend/issues/828

ghost avatar Aug 24 '21 16:08 ghost

nameserver03 uses AXFR, which is not cached, Cf zonemaster/zonemaster-engine#189 nameserver08 and nameserver09 use randomized case domain names, which prevent them to be cached. This randomization can be fixed if we use a fixed pattern for upper/lower cases, I'm not sure this randomization is mandatory. If I fix the problem for nameserver8 and nameserver9, I propose to close this issue. nameserver03 being covered with zonemaster/zonemaster-engine#189

vlevigneron avatar Nov 02 '21 13:11 vlevigneron

This randomization can be fixed if we use a fixed pattern for upper/lower cases, I'm not sure this randomization is mandatory. If I fix the problem for nameserver8 and nameserver9, I propose to close this issue. nameserver03 being covered with #189

For Nameserver08 it is mandatory, but not Nameserver09. I have created issue zonemaster/zonemaster#1009.

matsduf avatar Nov 02 '21 17:11 matsduf