update some comments and some printed messages

This commit is contained in:
Carson McManus 2022-12-05 10:36:38 -05:00
parent 2a5284f678
commit b5f4364733
2 changed files with 7 additions and 2 deletions

View file

@ -275,7 +275,7 @@ fn do_login_impl(
login.captcha_text = tui::prompt_captcha_text(&captcha_gid); login.captcha_text = tui::prompt_captcha_text(&captcha_gid);
} }
Err(LoginError::NeedEmail) => { Err(LoginError::NeedEmail) => {
println!("You should have received an email with a code."); println!("You should have received an email with a code. If you did not, check your spam folder, or abort and try again.");
print!("Enter code: "); print!("Enter code: ");
login.email_code = tui::prompt(); login.email_code = tui::prompt();
} }
@ -362,8 +362,10 @@ fn do_subcmd_setup(
username username
); );
} }
info!("Logging in to {}", username);
let session = do_login_raw(username).expect("Failed to log in. Account has not been linked."); let session = do_login_raw(username).expect("Failed to log in. Account has not been linked.");
info!("Adding authenticator...");
let mut linker = AccountLinker::new(session); let mut linker = AccountLinker::new(session);
let account: SteamGuardAccount; let account: SteamGuardAccount;
loop { loop {

View file

@ -62,6 +62,9 @@ impl AccountLinker {
return Err(AccountLinkError::AuthenticatorPresent); return Err(AccountLinkError::AuthenticatorPresent);
} }
2 => { 2 => {
// If the user has no phone number on their account, it will always return this status code.
// However, this does not mean that this status just means "no phone number". It can also
// be literally anything else, so that's why we return GenericFailure here.
return Err(AccountLinkError::GenericFailure); return Err(AccountLinkError::GenericFailure);
} }
1 => { 1 => {
@ -129,7 +132,7 @@ pub enum AccountLinkError {
MustConfirmEmail, MustConfirmEmail,
#[error("Authenticator is already present.")] #[error("Authenticator is already present.")]
AuthenticatorPresent, AuthenticatorPresent,
#[error("Steam was unable to link the authenticator to the account. No additional information about this error is available. This is a Steam error, not a steamguard-cli error. Try adding a phone number to your Steam account, or try again later.")] #[error("Steam was unable to link the authenticator to the account. No additional information about this error is available. This is a Steam error, not a steamguard-cli error. Try adding a phone number to your Steam account (which you can do here: https://store.steampowered.com/phone/add), or try again later.")]
GenericFailure, GenericFailure,
#[error(transparent)] #[error(transparent)]
Unknown(#[from] anyhow::Error), Unknown(#[from] anyhow::Error),