made identifiers case sensitive

questdb-confstr/README.md

@@ -13,7 +13,7 @@ service::key1=value1;key2=value2;key3=value3;
 
 A few rules:
 * The last semicolon is mandatory.
-* Service name and keys are case-insensitive.
+* Service name and keys are case-sensitive.
 * Keys are ASCII alphanumeric and can contain underscores.
 * Values are case-sensitive unicode strings which can contain any characters,
   * Except control characters (`0x00..=0x1f` and `0x7f..=0x9f`).

questdb-confstr/src/lib.rs

@@ -175,7 +175,7 @@ fn parse_ident(
     while let Some((pos, c)) = iter.peek0() {
         *next_pos = *pos;
         if c.is_ascii_alphanumeric() || *c == '_' {
-            token.push(c.to_ascii_lowercase());
+            token.push(*c);
             iter.next();
         } else {
             if token.is_empty() {

questdb-confstr/tests/tests.rs

@@ -55,9 +55,11 @@ fn basic() -> Result<(), ParsingError> {
 fn case_sensitivity() -> Result<(), ParsingError> {
     let input = "TcP::Host=LoCaLhOsT;Port=9000;";
     let config = parse_conf_str(input)?;
-    assert_eq!(config.service(), "tcp");
-    assert_eq!(config.get("host"), Some("LoCaLhOsT"));
-    assert_eq!(config.get("port"), Some("9000"));
+    assert_eq!(config.service(), "TcP");
+    assert_eq!(config.get("Host"), Some("LoCaLhOsT"));
+    assert_eq!(config.get("host"), None);
+    assert_eq!(config.get("Port"), Some("9000"));
+    assert_eq!(config.get("port"), None);
     Ok(())
 }
 
@@ -90,9 +92,9 @@ fn key_can_start_with_number() -> Result<(), ParsingError> {
 fn identifiers_can_contain_underscores() -> Result<(), ParsingError> {
     let input = "_A_::__x_Y__=42;";
     let config = parse_conf_str(input)?;
-    assert_eq!(config.service(), "_a_");
+    assert_eq!(config.service(), "_A_");
     let mut expected = HashMap::new();
-    expected.insert("__x_y__".to_string(), "42".to_string());
+    expected.insert("__x_Y__".to_string(), "42".to_string());
     Ok(())
 }
 
@@ -280,9 +282,9 @@ fn escaped_semicolon_missing_trailing() {
 fn escaped_semicolon() -> Result<(), ParsingError> {
     let input = "FTP::HOSTS=abc.com;;def.com;;ghi.net;PORTS=9000;;8000;;7000;;;";
     let config = parse_conf_str(input)?;
-    assert_eq!(config.service(), "ftp");
-    assert_eq!(config.get("hosts"), Some("abc.com;def.com;ghi.net"));
-    assert_eq!(config.get("ports"), Some("9000;8000;7000;"));
+    assert_eq!(config.service(), "FTP");
+    assert_eq!(config.get("HOSTS"), Some("abc.com;def.com;ghi.net"));
+    assert_eq!(config.get("PORTS"), Some("9000;8000;7000;"));
     Ok(())
 }