From 2907bd1953b15c81d4e7bfd1b060a9c9d947c2e4 Mon Sep 17 00:00:00 2001
From: Azriel Hoh <azriel91@gmail.com>
Date: Mon, 12 Nov 2018 10:09:30 +1300
Subject: Error when deserializing struct if encountering unknown fields.

Issue #225
---
 src/de.rs | 41 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 41 insertions(+)

(limited to 'src')

diff --git a/src/de.rs b/src/de.rs
index da30699..c7e8b34 100644
--- a/src/de.rs
+++ b/src/de.rs
@@ -178,6 +178,16 @@ enum ErrorKind {
     /// Dotted key attempted to extend something that is not a table.
     DottedKeyInvalidType,
 
+    /// An unexpected key was encountered.
+    ///
+    /// Used when deserializing a struct with a limited set of fields.
+    UnexpectedKeys {
+        /// The unexpected keys.
+        keys: Vec<String>,
+        /// Keys that may be specified.
+        available: &'static [&'static str],
+    },
+
     #[doc(hidden)]
     __Nonexhaustive,
 }
@@ -547,6 +557,28 @@ impl<'de> de::Deserializer<'de> for ValueDeserializer<'de> {
             }
         }
 
+        match &self.value.e {
+            E::InlineTable(values) | E::DottedTable(values) => {
+                let extra_fields = values.iter()
+                    .filter_map(|(key, _val)| {
+                        if !fields.contains(&&(**key)) {
+                            Some(key.clone())
+                        } else {
+                            None
+                        }
+                    })
+                    .collect::<Vec<Cow<'de, str>>>();
+
+                if !extra_fields.is_empty() {
+                    return Err(Error::from_kind(ErrorKind::UnexpectedKeys {
+                        keys: extra_fields.iter().map(|k| k.to_string()).collect::<Vec<_>>(),
+                        available: fields,
+                    }));
+                }
+            }
+            _ => {}
+        }
+
         if name == spanned::NAME && fields == &[spanned::START, spanned::END, spanned::VALUE] {
             let start = self.value.start;
             let end = self.value.end;
@@ -1653,6 +1685,14 @@ impl fmt::Display for Error {
             ErrorKind::DottedKeyInvalidType => {
                 "dotted key attempted to extend non-table type".fmt(f)?
             }
+            ErrorKind::UnexpectedKeys { ref keys, available } => {
+                write!(
+                    f,
+                    "unexpected keys in table: `{:?}`, available keys: `{:?}`",
+                    keys,
+                    available
+                )?
+            }
             ErrorKind::__Nonexhaustive => panic!(),
         }
 
@@ -1700,6 +1740,7 @@ impl error::Error for Error {
             ErrorKind::ExpectedTupleIndex { .. } => "expected table key",
             ErrorKind::ExpectedEmptyTable => "expected empty table",
             ErrorKind::DottedKeyInvalidType => "dotted key invalid type",
+            ErrorKind::UnexpectedKeys { .. } => "unexpected keys in table",
             ErrorKind::__Nonexhaustive => panic!(),
         }
     }
-- 
cgit v1.2.3