changed return type from option to result to display internal error

Author: SaschaScholly

src/gzip.rs

@@ -1,26 +1,25 @@
-pub fn decode(bytes: Vec<u8>) -> Option<String> {
+pub fn decode(bytes: Vec<u8>) -> Result<String, String> {
     use flate2::read::GzDecoder;
     use std::io::Read;
 
     let mut decoder = GzDecoder::new(&bytes[..]);
     let mut xml = String::new();
 
     match decoder.read_to_string(&mut xml) {
-        Ok(_) => Some(xml),
-        Err(_) => None,
+        Ok(_) => Ok(xml),
+        Err(error) => Err(error.to_string())
     }
 }
 
-pub fn encode(content: &str) -> Option<Vec<u8>> {
-    use flate2::read::GzEncoder;
-    use flate2::Compression;
+pub fn encode(content: &str) -> Result<String, String> {
+    use flate2::read::GzDecoder;
     use std::io::Read;
 
-    let mut encoder = GzEncoder::new(content.as_bytes(), Compression::best());
-    let mut bytes = Vec::new();
+    let mut decoder = GzDecoder::new(&bytes[..]);
+    let mut xml = String::new();
 
-    match encoder.read_to_end(&mut bytes) {
-        Ok(_) => Some(bytes),
-        Err(_) => None,
+    match decoder.read_to_string(&mut xml) {
+        Ok(_) => Ok(xml),
+        Err(error) => Err(error.to_string()),
     }
 }