diff --git a/src/securejoin.rs b/src/securejoin.rs index 64532beee..a0b7a9c97 100644 --- a/src/securejoin.rs +++ b/src/securejoin.rs @@ -484,19 +484,10 @@ pub(crate) async fn handle_securejoin_handshake( ) .await?; - let prefix = mime_message - .get_header(HeaderDef::SecureJoin) - .and_then(|step| step.get(..2)) - .unwrap_or("vc"); - // Alice -> Bob - send_alice_handshake_msg( - context, - autocrypt_contact_id, - &format!("{prefix}-auth-required"), - ) - .await - .context("failed sending auth-required handshake message")?; + send_alice_handshake_msg(context, autocrypt_contact_id, "vc-auth-required") + .await + .context("Failed sending auth-required handshake message")?; Ok(HandshakeMessage::Done) } SecureJoinStep::AuthRequired => { diff --git a/src/securejoin/bob.rs b/src/securejoin/bob.rs index fd9298562..a4530718c 100644 --- a/src/securejoin/bob.rs +++ b/src/securejoin/bob.rs @@ -27,7 +27,7 @@ use crate::tools::{smeared_time, time}; /// If Bob already has Alice's key, he sends `AUTH` token /// and forgets about the invite. /// If Bob does not yet have Alice's key, he sends `vc-request` -/// or `vg-request` message and stores a row in the `bobstate` table +/// message and stores a row in the `bobstate` table /// so he can check Alice's key against the fingerprint /// and send `AUTH` token later. /// @@ -301,14 +301,14 @@ pub(crate) async fn send_handshake_message( ) -> Result<()> { let mut msg = Message { viewtype: Viewtype::Text, - text: step.body_text(invite), + text: step.body_text(), hidden: true, ..Default::default() }; msg.param.set_cmd(SystemMessage::SecurejoinMessage); // Sends the step in Secure-Join header. - msg.param.set(Param::Arg, step.securejoin_header(invite)); + msg.param.set(Param::Arg, step.securejoin_header()); match step { BobHandshakeMsg::Request => { @@ -345,9 +345,9 @@ pub(crate) async fn send_handshake_message( /// Identifies the SecureJoin handshake messages Bob can send. pub(crate) enum BobHandshakeMsg { - /// vc-request or vg-request + /// vc-request Request, - /// vc-request-with-auth or vg-request-with-auth + /// vc-request-with-auth RequestWithAuth, } @@ -357,8 +357,8 @@ impl BobHandshakeMsg { /// This text has no significance to the protocol, but would be visible if users see /// this email message directly, e.g. when accessing their email without using /// DeltaChat. - fn body_text(&self, invite: &QrInvite) -> String { - format!("Secure-Join: {}", self.securejoin_header(invite)) + fn body_text(&self) -> String { + format!("Secure-Join: {}", self.securejoin_header()) } /// Returns the `Secure-Join` header value. @@ -366,18 +366,10 @@ impl BobHandshakeMsg { /// This identifies the step this message is sending information about. Most protocol /// steps include additional information into other headers, see /// [`send_handshake_message`] for these. - fn securejoin_header(&self, invite: &QrInvite) -> &'static str { + fn securejoin_header(&self) -> &'static str { match self { - Self::Request => match invite { - QrInvite::Contact { .. } => "vc-request", - QrInvite::Group { .. } => "vg-request", - QrInvite::Broadcast { .. } => "vg-request", - }, - Self::RequestWithAuth => match invite { - QrInvite::Contact { .. } => "vc-request-with-auth", - QrInvite::Group { .. } => "vg-request-with-auth", - QrInvite::Broadcast { .. } => "vg-request-with-auth", - }, + Self::Request => "vc-request", + Self::RequestWithAuth => "vc-request-with-auth", } } } @@ -447,7 +439,7 @@ async fn joining_chat_id( /// This has an `From for usize` impl yielding numbers between 0 and a 1000 /// which can be shown as a progress bar. pub(crate) enum JoinerProgress { - /// vg-vc-request-with-auth sent. + /// vc-request-with-auth sent. /// /// Typically shows as "alice@addr verified, introducing myself." RequestWithAuthSent, diff --git a/src/securejoin/securejoin_tests.rs b/src/securejoin/securejoin_tests.rs index 6c309089e..354960385 100644 --- a/src/securejoin/securejoin_tests.rs +++ b/src/securejoin/securejoin_tests.rs @@ -457,7 +457,7 @@ async fn test_secure_join() -> Result<()> { ); let msg = alice.parse_msg(&sent).await; assert!(!msg.was_encrypted()); - assert_eq!(msg.get_header(HeaderDef::SecureJoin).unwrap(), "vg-request"); + assert_eq!(msg.get_header(HeaderDef::SecureJoin).unwrap(), "vc-request"); assert!(msg.get_header(HeaderDef::SecureJoinInvitenumber).is_some()); assert!(!msg.header_exists(HeaderDef::AutoSubmitted)); @@ -478,7 +478,7 @@ async fn test_secure_join() -> Result<()> { assert!(msg.was_encrypted()); assert_eq!( msg.get_header(HeaderDef::SecureJoin).unwrap(), - "vg-auth-required" + "vc-auth-required" ); tcm.section("Step 4: Bob receives vg-auth-required, sends vg-request-with-auth"); @@ -516,7 +516,7 @@ async fn test_secure_join() -> Result<()> { assert!(msg.was_encrypted()); assert_eq!( msg.get_header(HeaderDef::SecureJoin).unwrap(), - "vg-request-with-auth" + "vc-request-with-auth" ); assert!(msg.get_header(HeaderDef::SecureJoinAuth).is_some()); let bob_fp = self_fingerprint(&bob).await?;