parsing

Cargo.lock

@@ -2,6 +2,21 @@
 # It is not intended for manual editing.
 version = 3
 
+[[package]]
+name = "addr2line"
+version = "0.17.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "b9ecd88a8c8378ca913a680cd98f0f13ac67383d35993f86c90a70e3f137816b"
+dependencies = [
+ "gimli",
+]
+
+[[package]]
+name = "adler"
+version = "1.0.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "f26201604c87b1e01bd3d98f8d5d9a8fcbb815e8cedb41ffccbeb4bf593a35fe"
+
 [[package]]
 name = "aho-corasick"
 version = "0.7.18"
@@ -61,6 +76,30 @@ name = "anyhow"
 version = "1.0.53"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "94a45b455c14666b85fc40a019e8ab9eb75e3a124e05494f5397122bc9eb06e0"
+dependencies = [
+ "backtrace",
+]
+
+[[package]]
+name = "autocfg"
+version = "1.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d468802bab17cbc0cc575e9b053f41e72aa36bfa6b7f55e3529ffa43161b97fa"
+
+[[package]]
+name = "backtrace"
+version = "0.3.64"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "5e121dee8023ce33ab248d9ce1493df03c3b38a659b240096fcbd7048ff9c31f"
+dependencies = [
+ "addr2line",
+ "cc",
+ "cfg-if",
+ "libc",
+ "miniz_oxide",
+ "object",
+ "rustc-demangle",
+]
 
 [[package]]
 name = "bitflags"
@@ -74,6 +113,12 @@ version = "1.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "c4872d67bab6358e59559027aa3b9157c53d9358c51423c17554809a8858e0f8"
 
+[[package]]
+name = "cc"
+version = "1.0.72"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "22a9137b95ea06864e018375b72adfb7db6e6f68cfc8df5a04d00288050485ee"
+
 [[package]]
 name = "cfg-if"
 version = "1.0.0"
@@ -97,6 +142,12 @@ dependencies = [
  "wasi",
 ]
 
+[[package]]
+name = "gimli"
+version = "0.26.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "78cc372d058dcf6d5ecd98510e7fbc9e5aec4d21de70f65fea8fecebcd881bd4"
+
 [[package]]
 name = "heck"
 version = "0.4.0"
@@ -178,6 +229,16 @@ version = "0.2.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "68354c5c6bd36d73ff3feceb05efa59b6acb7626617f4962be322a825e61f79a"
 
+[[package]]
+name = "miniz_oxide"
+version = "0.4.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "a92518e98c078586bc6c934028adcca4c92a53d6a958196de835170a01d84e4b"
+dependencies = [
+ "adler",
+ "autocfg",
+]
+
 [[package]]
 name = "mio"
 version = "0.7.14"
@@ -230,6 +291,15 @@ dependencies = [
  "libc",
 ]
 
+[[package]]
+name = "object"
+version = "0.27.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "67ac1d3f9a1d3616fd9a60c8d74296f22406a238b6a72f5cc1e6f314df4ffbf9"
+dependencies = [
+ "memchr",
+]
+
 [[package]]
 name = "once_cell"
 version = "1.9.0"
@@ -357,6 +427,12 @@ version = "0.6.25"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "f497285884f3fcff424ffc933e56d7cbca511def0c9831a7f9b5f6153e3cc89b"
 
+[[package]]
+name = "rustc-demangle"
+version = "0.1.21"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "7ef03e0a2b150c7a90d01faf6254c9c48a41e95fb2a8c2ac1c6f0d2b9aefc342"
+
 [[package]]
 name = "scopeguard"
 version = "1.1.0"

amqp_transport/Cargo.toml

@@ -6,7 +6,7 @@ edition = "2021"
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
 [dependencies]
-anyhow = "1.0.53"
+anyhow = { version = "1.0.53", features = ["backtrace"] }
 nom = "7.1.0"
 once_cell = "1.9.0"
 rand = "0.8.4"

amqp_transport/src/classes/mod.rs

@@ -46,3 +46,31 @@ pub fn parse_method(payload: &[u8]) -> Result<generated::Class, TransError> {
         Err(nom::Err::Failure(err) | nom::Err::Error(err)) => Err(err),
     }
 }
+
+#[cfg(test)]
+mod tests {
+    #[test]
+    fn pack_few_bits() {
+        let bits = [true, false, true];
+
+        let mut buffer = [0u8; 2];
+        super::write_helper::bit(&bits, &mut buffer.as_mut_slice()).unwrap();
+
+        let (_, parsed_bits) = super::parse_helper::bit(&buffer, 3).unwrap();
+        assert_eq!(bits.as_slice(), parsed_bits.as_slice());
+    }
+
+    #[test]
+    fn pack_many_bits() {
+        let bits = [
+            /* first 8 */
+            true, true, true, true, false, false, false, false, /* second 4 */
+            true, false, true, true,
+        ];
+        let mut buffer = [0u8; 2];
+        super::write_helper::bit(&bits, &mut buffer.as_mut_slice()).unwrap();
+
+        let (_, parsed_bits) = super::parse_helper::bit(&buffer, 12).unwrap();
+        assert_eq!(bits.as_slice(), parsed_bits.as_slice());
+    }
+}

amqp_transport/src/classes/parse_helper.rs

@@ -3,14 +3,20 @@ use crate::classes::generated::{
     Bit, Long, Longlong, Longstr, Octet, Short, Shortstr, Table, Timestamp,
 };
 use crate::error::{ConException, ProtocolError, TransError};
+use nom::branch::alt;
+use nom::bytes::complete::{tag, take};
 use nom::error::ErrorKind;
+use nom::multi::count;
+use nom::number::complete::{f32, f64, i16, i32, i64, i8, u16, u32, u64, u8};
+use nom::number::Endianness::Big;
+use std::collections::HashMap;
 
-impl nom::error::ParseError<&[u8]> for TransError {
+impl<T> nom::error::ParseError<T> for TransError {
-    fn from_error_kind(_input: &[u8], _kind: ErrorKind) -> Self {
+    fn from_error_kind(_input: T, _kind: ErrorKind) -> Self {
         ProtocolError::ConException(ConException::SyntaxError).into()
     }
 
-    fn append(_input: &[u8], _kind: ErrorKind, other: Self) -> Self {
+    fn append(_input: T, _kind: ErrorKind, other: Self) -> Self {
         other
     }
 }
@@ -25,33 +31,167 @@ macro_rules! fail {
     };
 }
 
+use crate::classes::{FieldValue, TableFieldName};
 pub use fail;
 
 pub fn octet(input: &[u8]) -> IResult<Octet> {
-    todo!()
+    u8(input)
 }
+
 pub fn short(input: &[u8]) -> IResult<Short> {
-    todo!()
+    u16(Big)(input)
 }
+
 pub fn long(input: &[u8]) -> IResult<Long> {
-    todo!()
+    u32(Big)(input)
 }
+
 pub fn longlong(input: &[u8]) -> IResult<Longlong> {
-    todo!()
+    u64(Big)(input)
 }
-// todo: doing this using a vec is a bit wasteful, consider not doing that
+
-pub fn bit(input: &[u8], amount: u8) -> IResult<Vec<Bit>> {
+pub fn bit(input: &[u8], amount: usize) -> IResult<Vec<Bit>> {
-    todo!()
+    let octets = (amount + 7) / 8;
+    let (input, bytes) = take(octets)(input)?;
+
+    let mut vec = Vec::new();
+    let mut byte_index = 0;
+    let mut total_index = 0;
+
+    for &byte in bytes {
+        while byte_index < 8 && total_index < amount {
+            let next_bit = 1 & (byte >> byte_index);
+            let bit_bool = match next_bit {
+                0 => false,
+                1 => true,
+                _ => unreachable!(),
+            };
+            vec.push(bit_bool);
+            byte_index += 1;
+            total_index += 1;
+        }
+        byte_index = 0;
+    }
+
+    Ok((input, vec))
 }
+
 pub fn shortstr(input: &[u8]) -> IResult<Shortstr> {
-    todo!()
+    let (input, len) = u8(input)?;
+    let (input, str_data) = take(usize::from(len))(input)?;
+    let data = String::from_utf8(str_data.into()).map_err(|_| {
+        nom::Err::Failure(ProtocolError::ConException(ConException::SyntaxError).into())
+    })?;
+    Ok((input, data))
 }
+
 pub fn longstr(input: &[u8]) -> IResult<Longstr> {
-    todo!()
+    let (input, len) = u32(Big)(input)?;
+    let (input, str_data) = take(usize::try_from(len).unwrap())(input)?;
+    let data = str_data.into();
+    Ok((input, data))
 }
+
 pub fn timestamp(input: &[u8]) -> IResult<Timestamp> {
-    todo!()
+    u64(Big)(input)
 }
+
 pub fn table(input: &[u8]) -> IResult<Table> {
-    todo!()
+    let (input, len) = u32(Big)(input)?;
+
+    let (input, values) = count(table_value_pair, usize::try_from(len).unwrap())(input)?;
+    let table = HashMap::from_iter(values.into_iter());
+    Ok((input, table))
+}
+
+fn table_value_pair(input: &[u8]) -> IResult<(TableFieldName, FieldValue)> {
+    let (input, field_name) = shortstr(input)?;
+    let (input, field_value) = field_value(input)?;
+    Ok((input, (field_name, field_value)))
+}
+
+fn field_value(input: &[u8]) -> IResult<FieldValue> {
+    type R<'a> = IResult<'a, FieldValue>;
+
+    fn boolean(input: &[u8]) -> R {
+        let (input, _) = tag(b"t")(input)?;
+        let (input, bool_byte) = u8(input)?;
+        match bool_byte {
+            0 => Ok((input, FieldValue::Boolean(false))),
+            1 => Ok((input, FieldValue::Boolean(true))),
+            _ => fail!(),
+        }
+    }
+
+    macro_rules! number {
+        ($tag:literal, $name:ident, $comb:expr, $value:ident, $r:path) => {
+            fn $name(input: &[u8]) -> $r {
+                let (input, _) = tag($tag)(input)?;
+                $comb(input).map(|(input, int)| (input, FieldValue::$value(int)))
+            }
+        };
+    }
+
+    number!(b"b", short_short_int, i8, ShortShortInt, R);
+    number!(b"B", short_short_uint, u8, ShortShortUInt, R);
+    number!(b"U", short_int, i16(Big), ShortInt, R);
+    number!(b"u", short_uint, u16(Big), ShortUInt, R);
+    number!(b"I", long_int, i32(Big), LongInt, R);
+    number!(b"i", long_uint, u32(Big), LongUInt, R);
+    number!(b"L", long_long_int, i64(Big), LongLongInt, R);
+    number!(b"l", long_long_uint, u64(Big), LongLongUInt, R);
+    number!(b"f", float, f32(Big), Float, R);
+    number!(b"d", double, f64(Big), Double, R);
+
+    fn decimal(input: &[u8]) -> R {
+        let (input, _) = tag("D")(input)?;
+        let (input, scale) = u8(input)?;
+        let (input, value) = u32(Big)(input)?;
+        Ok((input, FieldValue::DecimalValue(scale, value)))
+    }
+
+    fn short_str(input: &[u8]) -> R {
+        let (input, _) = tag("s")(input)?;
+        let (input, str) = shortstr(input)?;
+        Ok((input, FieldValue::ShortString(str)))
+    }
+
+    fn long_str(input: &[u8]) -> R {
+        let (input, _) = tag("S")(input)?;
+        let (input, str) = longstr(input)?;
+        Ok((input, FieldValue::LongString(str)))
+    }
+
+    fn field_array(input: &[u8]) -> R {
+        let (input, _) = tag("A")(input)?;
+        // todo is it i32?
+        let (input, len) = u32(Big)(input)?;
+        count(field_value, usize::try_from(len).unwrap())(input)
+            .map(|(input, value)| (input, FieldValue::FieldArray(value)))
+    }
+
+    number!(b"T", timestamp, u64(Big), Timestamp, R);
+
+    fn field_table(input: &[u8]) -> R {
+        table(input).map(|(input, value)| (input, FieldValue::FieldTable(value)))
+    }
+
+    alt((
+        boolean,
+        short_short_int,
+        short_short_uint,
+        short_int,
+        short_uint,
+        long_int,
+        long_uint,
+        long_long_int,
+        long_long_uint,
+        float,
+        double,
+        decimal,
+        short_str,
+        long_str,
+        field_array,
+        timestamp,
+    ))(input)
 }

amqp_transport/src/connection.rs

@@ -1,5 +1,5 @@
 use crate::classes::FieldValue;
-use crate::error::{ProtocolError, Result};
+use crate::error::{ConException, ProtocolError, Result};
 use crate::frame::{Frame, FrameType};
 use crate::{classes, frame};
 use anyhow::Context;
@@ -54,7 +54,7 @@ impl Connection {
                     .local_addr()
                     .context("failed to get local_addr")?,
             ),
-            mechanisms: "none".to_string().into(),
+            mechanisms: "PLAIN".to_string().into(),
             locales: "en_US".to_string().into(),
         });
 
@@ -72,6 +72,17 @@ impl Connection {
         )
         .await?;
 
+        let start_ok_frame = frame::read_frame(&mut self.stream, self.max_frame_size).await?;
+        debug!(?start_ok_frame, "Received Start-Ok frame");
+
+        if start_ok_frame.kind != FrameType::Method {
+            return Err(ProtocolError::ConException(ConException::Todo).into());
+        }
+
+        let class = classes::parse_method(&start_ok_frame.payload)?;
+
+        debug!(?class, "extracted method");
+
         Ok(())
     }
 
@@ -120,6 +131,8 @@ fn server_properties(host: SocketAddr) -> classes::Table {
         FieldValue::LongString(host_str.into())
     };
 
+    // todo: fix
+
     //HashMap::from([
     //    ("host".to_string(), host_value),
     //    ("product".to_string(), ss("no name yet")),

amqp_transport/src/error.rs

@@ -38,6 +38,8 @@ pub enum ConException {
     SyntaxError,
     #[error("504 Channel error")]
     ChannelError,
+    #[error("xxx Not decided yet")]
+    Todo,
 }
 
 #[derive(Debug, thiserror::Error)]