make data-encoding a required dependency for proto

CHANGELOG.md

@@ -9,6 +9,7 @@ All notes should be prepended with the location of the change, e.g. `(proto)` or
 
 ### Changed
 
+- (proto) `data-encoding` is now a required dependency #1208
 - (all) minimum rustc version now `1.42`
 - (resolver) For all NxDomain and NoError/NoData responses, `ResolveErrorKind::NoRecordsFound` will be returned #1197
 - (server) Support for lowercase DNSClass and RecordType fields in zonefiles (@zhanif3) #1186

crates/proto/Cargo.toml

@@ -38,7 +38,7 @@ maintenance = { status = "actively-developed" }
 [features]
 dnssec-openssl = ["dnssec", "openssl"]
 dnssec-ring = ["dnssec", "ring"]
-dnssec = ["data-encoding"]
+dnssec = []
 testing = []
 tokio-runtime = ["tokio/rt-core", "tokio/udp", "tokio/tcp", "tokio/time"]
 default = ["tokio-runtime"]
@@ -60,7 +60,7 @@ path = "src/lib.rs"
 [dependencies]
 async-trait = "0.1.36"
 backtrace = "0.3.50"
-data-encoding = { version = "2.2.0", optional = true }
+data-encoding = "2.2.0"
 enum-as-inner = "0.3"
 futures-channel = { version = "0.3.5", default-features = false, features = ["std"] }
 futures-io = { version = "0.3.5", default-features = false, features = ["std"] }