f Disallow empty keys

Author: tnull

src/io/sqlite_store.rs

@@ -48,7 +48,7 @@ impl SqliteStore {
 		let sql = format!(
 			"CREATE TABLE IF NOT EXISTS {} (
 			namespace TEXT NOT NULL,
-			key TEXT NOT NULL,
+			key TEXT NOT NULL CHECK (length(key) > 0),
 			value BLOB, PRIMARY KEY ( namespace, key )
 		);",
 			KV_TABLE_NAME
@@ -215,6 +215,10 @@ mod tests {
 			// Test the basic KVStore operations.
 			sqlite_store.write(namespace, key, &data).unwrap();
 
+			// Test empty namespace is allowed, but not empty key.
+			sqlite_store.write("", key, &data).unwrap();
+			assert!(sqlite_store.write(namespace, "", &data).is_err());
+
 			let listed_keys = sqlite_store.list(namespace).unwrap();
 			assert_eq!(listed_keys.len(), 1);
 			assert_eq!(listed_keys[0], key);