namecoin / ncprop279

Bridge between Tor Prop279 (Pluggable Naming) clients and Namecoin.
https://www.namecoin.org/
GNU General Public License v3.0
1 stars 5 forks source link

Add integration tests for uppercase domains #13

Open JeremyRand opened 4 years ago

JeremyRand commented 4 years ago

madns converts all domain names to lowercase before feeding them to ncdns: https://github.com/hlandau/madns/blob/26979b3e4b5aa3e0bd53cf0a014f9eaf43b578e3/madns.go#L126

We should add an integration test to ncprop279 that will detect if this conversion doesn't work as intended, e.g. when passed uppercase names.

Thanks to xa55 on Reddit for pointing out that this is important enough to merit an integration test.