fix: "Explore" (using --default-profile)

Author: dpc

crates/rostra-client/src/task/request_handler.rs

@@ -113,10 +113,10 @@ impl RequestHandler {
             match err {
                 // normal, mostly ignore
                 IncomingConnectionError::Connection { source: _, .. } => {
-                    trace!(target: LOG_TARGET, err=%err.fmt_compact(), %peer_addr, "Client disconnected");
+                    trace!(target: LOG_TARGET, err = %err.fmt_compact(), %peer_addr, "Client disconnected");
                 }
                 _ => {
-                    debug!(target: LOG_TARGET, err=%err.fmt_compact(), %peer_addr, "Error handling incoming connection");
+                    debug!(target: LOG_TARGET, err = %err.fmt_compact(), %peer_addr, "Error handling incoming connection");
                 }
             }
         }

crates/rostra-web-ui/src/error.rs

@@ -9,7 +9,7 @@ use rostra_client_db::DbError;
 use rostra_util_error::{BoxedError, FmtCompact as _};
 use serde::Serialize;
 use snafu::Snafu;
-use tracing::info;
+use tracing::{debug, warn};
 
 use super::routes::AppJson;
 use crate::{LOG_TARGET, UiStateClientError};
@@ -78,6 +78,7 @@ pub enum RequestError {
     #[snafu(visibility(pub(crate)))]
     Other { source: BoxedError },
     #[snafu(visibility(pub(crate)))]
+    #[snafu(display("InternalServerError: {msg}"))]
     InternalServerError { msg: &'static str },
     #[snafu(visibility(pub(crate)))]
     LoginRequired { redirect: Option<String> },
@@ -92,10 +93,9 @@ pub type RequestResult<T> = std::result::Result<T, RequestError>;
 
 impl IntoResponse for RequestError {
     fn into_response(self) -> Response {
-        info!(
+        debug!(
             target: LOG_TARGET,
-
-            err=%self.fmt_compact(),
+            err = %self.fmt_compact(),
             "Request Error"
         );
 
@@ -118,10 +118,17 @@ impl IntoResponse for RequestError {
                         };
                         return Redirect::to(&url).into_response();
                     }
-                    _ => (
-                        StatusCode::INTERNAL_SERVER_ERROR,
-                        "Internal Service Error".to_owned(),
-                    ),
+                    err => {
+                        warn!(
+                            target: LOG_TARGET,
+                            err = %err.fmt_compact(),
+                            "Unexpected Request Error"
+                        );
+                        (
+                            StatusCode::INTERNAL_SERVER_ERROR,
+                            "Internal Service Error".to_owned(),
+                        )
+                    }
                 }
             }
         };

crates/rostra-web-ui/src/routes/unlock.rs

@@ -117,15 +117,15 @@ pub async fn post_unlock(
         }
     };
 
-    // 2. Insert session data into store (this creates/updates the session)
+    // 2. Insert session data and save to store
     session
         .insert(SESSION_KEY, &UserSessionData::new(rostra_id))
         .await
         .boxed()
         .context(OtherSnafu)?;
+    session.save().await.boxed().context(OtherSnafu)?;
 
-    // 3. Get session ID (now available after insert) and store secret
-    // The session ID is available after the session has been modified.
+    // 3. Get session ID (now available after save) and store secret
     if let Some(session_token) = SessionToken::from_session(&session) {
         state.set_session_secret(session_token, secret_key_opt);
 

crates/rostra-web-ui/src/routes/unlock/session.rs

@@ -121,19 +121,25 @@ impl FromRequestParts<Arc<UiState>> for UserSession {
                         .build()
                     })?;
 
-                    // Insert session data first
+                    // Insert session data and save to store
                     let data = UserSessionData::new(default_id);
                     session.insert(SESSION_KEY, &data).await.map_err(|_| {
                         InternalServerSnafu {
                             msg: "failed to insert session",
                         }
                         .build()
                     })?;
+                    session.save().await.map_err(|_| {
+                        InternalServerSnafu {
+                            msg: "failed to save session",
+                        }
+                        .build()
+                    })?;
 
-                    // Now get the session token (available after insert)
+                    // Now get the session token (available after save)
                     let session_token = SessionToken::from_session(&session).ok_or_else(|| {
                         InternalServerSnafu {
-                            msg: "session has no ID after insert",
+                            msg: "session has no ID after save",
                         }
                         .build()
                     })?;