Skip to content

Commit

Permalink
uds: cleanup error reporting to user
Browse files Browse the repository at this point in the history
  • Loading branch information
colemickens committed Aug 5, 2024
1 parent 5898cf2 commit 6d64a9b
Showing 1 changed file with 13 additions and 6 deletions.
19 changes: 13 additions & 6 deletions src/cli/cmd/login/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,6 @@ pub async fn dnixd_uds() -> color_eyre::Result<SendRequest<axum::body::Body>> {
let response = sender.send_request(request).await?;

if response.status() != StatusCode::OK {
tracing::error!("failed to connect to determinate-nixd socket");
return Err(eyre!("failed to connect to determinate-nixd socket"));
}

Expand All @@ -90,10 +89,15 @@ impl LoginSubcommand {
let dnixd_uds = match dnixd_uds().await {
Ok(socket) => Some(socket),
Err(err) => {
tracing::error!(
"failed to connect to determinate-nixd socket, will not attempt to use it: {:?}",
err
if tracing::enabled!(tracing::Level::DEBUG) {
tracing::debug!(
"failed to connect to determinate-nixd socket, will not attempt to use it: {}", err
);
} else {
tracing::warn!(
"failed to connect to determinate-nixd socket, will not attempt to use it."
);
}
None
}
};
Expand Down Expand Up @@ -199,8 +203,6 @@ impl LoginSubcommand {
let bytes = body.collect().await.unwrap_or_default().to_bytes();
let text: String = String::from_utf8_lossy(&bytes).into();

tracing::trace!("sent the add request: {:?}", text);

token_updated = true;
}

Expand All @@ -209,6 +211,11 @@ impl LoginSubcommand {
"failed to update netrc via determinatenixd, falling back to local-file approach"
);

// check if user is root or not
if !nix::unistd::Uid::effective().is_root() {
return Err(eyre!("`fh login` is attempting to update a file owned by root, please re-run the same command, prefixed with `sudo -i`."));
}

update_netrc_file(&netrc_file_path, &netrc_contents).await?;

// only update user_nix_config if we could not use determinatenixd
Expand Down

0 comments on commit 6d64a9b

Please sign in to comment.