fix(confirm): fix confirmations not working because of changes to /mobileconf/multiajaxop endpoint (#439)

fixes #438

Author: dyc3

src/commands/confirm.rs

@@ -74,12 +74,12 @@ where
 			let mut any_failed = false;
 
 			fn submit_loop(
-				f: impl Fn() -> Result<(), ConfirmerError>,
+				submit: impl Fn() -> Result<(), ConfirmerError>,
 				fail_fast: bool,
 			) -> Result<(), ConfirmerError> {
 				let mut attempts = 0;
 				loop {
-					match f() {
+					match submit() {
 						Ok(_) => break,
 						Err(ConfirmerError::InvalidTokens) => {
 							error!("Invalid tokens, but they should be valid already. This is weird, stopping.");
@@ -116,7 +116,7 @@ where
 			if self.accept_all {
 				info!("accepting all confirmations");
 				match submit_loop(
-					|| confirmer.accept_confirmations(&confirmations),
+					|| confirmer.accept_confirmations_bulk(&confirmations),
 					self.fail_fast,
 				) {
 					Ok(_) => {}
@@ -130,7 +130,10 @@ where
 				}
 			} else if std::io::stdout().is_tty() {
 				let (accept, deny) = tui::prompt_confirmation_menu(confirmations)?;
-				match submit_loop(|| confirmer.accept_confirmations(&accept), self.fail_fast) {
+				match submit_loop(
+					|| confirmer.accept_confirmations_bulk(&accept),
+					self.fail_fast,
+				) {
 					Ok(_) => {}
 					Err(err) => {
 						warn!("accept confirmation result: {}", err);
@@ -140,7 +143,7 @@ where
 						any_failed = true;
 					}
 				}
-				match submit_loop(|| confirmer.deny_confirmations(&deny), self.fail_fast) {
+				match submit_loop(|| confirmer.deny_confirmations_bulk(&deny), self.fail_fast) {
 					Ok(_) => {}
 					Err(err) => {
 						warn!("deny confirmation result: {}", err);

steamguard/src/confirmation.rs

@@ -147,6 +147,7 @@ where
 			)
 			.header(USER_AGENT, "steamguard-cli")
 			.header(COOKIE, cookies.cookies(&STEAM_COOKIE_URL).unwrap())
+			.header("Origin", "https://steamcommunity.com")
 			.query(&query_params)
 			.send()?;
 
@@ -226,6 +227,7 @@ where
 				CONTENT_TYPE,
 				"application/x-www-form-urlencoded; charset=UTF-8",
 			)
+			.header("Origin", "https://steamcommunity.com")
 			.body(query_params)
 			.send()?;
 
@@ -255,11 +257,39 @@ where
 		Ok(())
 	}
 
+	/// Bulk accept confirmations.
+	///
+	/// Sends one request per confirmation.
 	pub fn accept_confirmations(&self, confs: &[Confirmation]) -> Result<(), ConfirmerError> {
-		self.send_multi_confirmation_ajax(confs, ConfirmationAction::Accept)
+		for conf in confs {
+			self.accept_confirmation(conf)?;
+		}
+
+		Ok(())
 	}
 
+	/// Bulk deny confirmations.
+	///
+	/// Sends one request per confirmation.
 	pub fn deny_confirmations(&self, confs: &[Confirmation]) -> Result<(), ConfirmerError> {
+		for conf in confs {
+			self.deny_confirmation(conf)?;
+		}
+
+		Ok(())
+	}
+
+	/// Bulk accept confirmations.
+	///
+	/// Uses a different endpoint than `accept_confirmation()` to submit multiple confirmations in one request.
+	pub fn accept_confirmations_bulk(&self, confs: &[Confirmation]) -> Result<(), ConfirmerError> {
+		self.send_multi_confirmation_ajax(confs, ConfirmationAction::Accept)
+	}
+
+	/// Bulk deny confirmations.
+	///
+	/// Uses a different endpoint than `deny_confirmation()` to submit multiple confirmations in one request.
+	pub fn deny_confirmations_bulk(&self, confs: &[Confirmation]) -> Result<(), ConfirmerError> {
 		self.send_multi_confirmation_ajax(confs, ConfirmationAction::Deny)
 	}