chore: unwrap

Author: ovr

rust/cubesql/cubesql/src/compile/router.rs

@@ -456,7 +456,6 @@ impl QueryRouter {
                     .session_manager
                     .server
                     .auth
-                    // TODO do we want to send actual password here?
                     .authenticate(sql_auth_request, Some(to_user.clone()), None)
                     .await
                     .map_err(|e| {
@@ -466,8 +465,12 @@ impl QueryRouter {
                     .set_auth_context(Some(authenticate_response.context));
             } else {
                 return Err(CompilationError::user(format!(
-                    "user '{:?}' is not allowed to switch to '{}'",
-                    auth_context.user(),
+                    "user '{}' is not allowed to switch to '{}'",
+                    auth_context
+                        .user()
+                        .as_ref()
+                        .map(|v| v.as_str())
+                        .unwrap_or("not specified"),
                     to_user
                 )));
             }

rust/cubesql/cubesql/src/compile/snapshots/cubesql__compile__tests__set_bad_user.snap

@@ -2,4 +2,4 @@
 source: cubesql/src/compile/mod.rs
 expression: "execute_queries_with_flags(vec![\"SET user = 'bad_user'\".to_string()],\nDatabaseProtocol::PostgreSQL).await.err().unwrap().to_string()"
 ---
-Error during planning: SQLCompilationError: User: user 'None' is not allowed to switch to 'bad_user'
+Error during planning: SQLCompilationError: User: user 'not specified' is not allowed to switch to 'bad_user'