.gitignore

@@ -1,4 +1,3 @@
 /target
 .env
 result
-**/.DS_Store

Cargo.lock

@@ -839,11 +839,9 @@ dependencies = [
  "enum_stringify",
  "envconfig",
  "futures",
- "log",
  "mongodb",
  "serde",
  "teloxide",
- "thiserror 2.0.12",
  "tokio",
 ]
 

Cargo.toml

@@ -14,13 +14,7 @@ dotenvy = "0.15.7"
 enum_stringify = "0.6.3"
 envconfig = "0.11.0"
 futures = "0.3.31"
-log = "0.4.27"
 mongodb = "3.2.3"
 serde = { version = "1.0.219", features = ["derive", "serde_derive"] }
 teloxide = { version = "0.14.0", features = ["macros", "postgres-storage-nativetls"] }
-thiserror = "2.0.12"
 tokio = { version = "1.44.1", features = ["rt-multi-thread", "macros"] }
-
-[lints.clippy]
-print_stdout = "warn"
-unwrap_used = "warn"

src/admin.rs

@@ -3,12 +3,8 @@ use teloxide::{
     utils::{command::BotCommands, render::RenderMessageTextHelper},
 };
 
+use crate::db::{CallDB, DB};
 use crate::LogMsg;
-use crate::{
-    db::{CallDB, DB},
-    BotResult,
-};
-use log::info;
 
 // These are should not appear in /help
 #[derive(BotCommands, Clone)]
@@ -34,15 +30,9 @@ pub async fn admin_command_handler(
     bot: Bot,
     msg: Message,
     cmd: AdminCommands,
-) -> BotResult<()> {
-    let tguser = match msg.from.clone() {
-        Some(user) => user,
-        None => return Ok(()), // do nothing, cause its not usecase of function
-    };
-    info!(
-        "MSG: {}",
-        msg.html_text().unwrap_or("|EMPTY_MESSAGE|".into())
-    );
+) -> Result<(), teloxide::RequestError> {
+    let tguser = msg.from.clone().unwrap();
+    println!("MSG: {}", msg.html_text().unwrap());
     match cmd {
         AdminCommands::MyId => {
             bot.send_message(msg.chat.id, format!("Your ID is: {}", tguser.id))
@@ -64,7 +54,7 @@ pub async fn admin_command_handler(
             Ok(())
         }
         AdminCommands::Deop => {
-            db.set_admin(tguser.id.0 as i64, false).await?;
+            db.set_admin(tguser.id.0 as i64, false).await;
             bot.send_message(msg.chat.id, "You are not an admin anymore")
                 .await?;
             Ok(())
@@ -79,27 +69,22 @@ pub async fn secret_command_handler(
     msg: Message,
     cmd: SecretCommands,
     admin_password: String,
-) -> BotResult<()> {
-    info!("Admin Pass: {}", admin_password);
-    let tguser = match msg.from.clone() {
-        Some(user) => user,
-        None => return Ok(()), // do nothing, cause its not usecase of function
-    };
+) -> Result<(), teloxide::RequestError> {
+    println!("Admin Pass: {}", admin_password);
+    let tguser = msg.from.clone().unwrap();
     let user = db
         .get_or_init_user(tguser.id.0 as i64, &tguser.first_name)
-        .await?;
-    info!(
-        "MSG: {}",
-        msg.html_text().unwrap_or("|EMPTY_MESSAGE|".into())
-    );
+        .await;
+    println!("MSG: {}", msg.html_text().unwrap());
     match cmd {
         SecretCommands::Secret { pass } => {
-            if user.is_admin {
-                bot.send_message(tguser.id, "You are an admin already")
+            if user.is_admin == true {
+                bot.send_message(msg.from.unwrap().id, "You are an admin already")
                     .await?;
             } else if pass == admin_password {
-                db.set_admin(user.id, true).await?;
-                bot.send_message(tguser.id, "You are an admin now!").await?;
+                db.set_admin(user.id, true).await;
+                bot.send_message(msg.from.unwrap().id, "You are an admin now!")
+                    .await?;
             }
             Ok(())
         }

src/db/mod.rs

@@ -1,7 +1,7 @@
 use async_trait::async_trait;
 use chrono::{DateTime, Utc};
 use enum_stringify::EnumStringify;
-use futures::stream::TryStreamExt;
+use futures::stream::{StreamExt, TryStreamExt};
 
 use mongodb::options::IndexOptions;
 use mongodb::{bson::doc, options::ClientOptions, Client};
@@ -38,7 +38,7 @@ pub struct User {
 macro_rules! query_call {
     ($func_name:ident, $self:ident, $db:ident, $return_type:ty, $body:block) => {
         pub async fn $func_name<D: CallDB>(&$self, $db: &mut D)
-            -> DbResult<$return_type> $body
+            -> Result<$return_type, Box<dyn std::error::Error>> $body
     };
 }
 
@@ -101,14 +101,14 @@ pub struct DB {
 }
 
 impl DB {
-    pub async fn new<S: Into<String>>(db_url: S) -> DbResult<Self> {
-        let options = ClientOptions::parse(db_url.into()).await?;
-        let client = Client::with_options(options)?;
+    pub async fn new<S: Into<String>>(db_url: S) -> Self {
+        let options = ClientOptions::parse(db_url.into()).await.unwrap();
+        let client = Client::with_options(options).unwrap();
 
-        Ok(DB { client })
+        DB { client }
     }
 
-    pub async fn migrate(&mut self) -> DbResult<()> {
+    pub async fn migrate(&mut self) -> Result<(), mongodb::error::Error> {
         let events = self.get_database().await.collection::<Event>("events");
         events
             .create_index(
@@ -121,8 +121,8 @@ impl DB {
         Ok(())
     }
 
-    pub async fn init<S: Into<String>>(db_url: S) -> DbResult<Self> {
-        let mut db = Self::new(db_url).await?;
+    pub async fn init<S: Into<String>>(db_url: S) -> Result<Self, mongodb::error::Error> {
+        let mut db = Self::new(db_url).await;
         db.migrate().await?;
 
         Ok(db)
@@ -136,22 +136,24 @@ impl CallDB for DB {
     }
 }
 
-pub type DbError = mongodb::error::Error;
-pub type DbResult<T> = Result<T, DbError>;
-
 #[async_trait]
 pub trait CallDB {
     //type C;
     async fn get_database(&mut self) -> Database;
     //async fn get_pool(&mut self) -> PooledConnection<'_, AsyncDieselConnectionManager<C>>;
-    async fn get_users(&mut self) -> DbResult<Vec<User>> {
+    async fn get_users(&mut self) -> Vec<User> {
         let db = self.get_database().await;
         let users = db.collection::<User>("users");
-
-        users.find(doc! {}).await?.try_collect().await
+        users
+            .find(doc! {})
+            .await
+            .unwrap()
+            .map(|u| u.unwrap())
+            .collect()
+            .await
     }
 
-    async fn set_admin(&mut self, userid: i64, isadmin: bool) -> DbResult<()> {
+    async fn set_admin(&mut self, userid: i64, isadmin: bool) {
         let db = self.get_database().await;
         let users = db.collection::<User>("users");
         users
@@ -163,12 +165,11 @@ pub trait CallDB {
                     "$set": { "is_admin": isadmin }
                 },
             )
-            .await?;
-
-        Ok(())
+            .await
+            .unwrap();
     }
 
-    async fn get_or_init_user(&mut self, userid: i64, firstname: &str) -> DbResult<User> {
+    async fn get_or_init_user(&mut self, userid: i64, firstname: &str) -> User {
         let db = self.get_database().await;
         let users = db.collection::<User>("users");
 
@@ -181,15 +182,21 @@ pub trait CallDB {
                 },
             )
             .upsert(true)
-            .await?;
+            .await
+            .unwrap();
 
-        Ok(users
+        users
             .find_one(doc! { "id": userid })
-            .await?
-            .expect("no such user created"))
+            .await
+            .unwrap()
+            .expect("no such user created")
     }
 
-    async fn get_message(&mut self, chatid: i64, messageid: i32) -> DbResult<Option<Message>> {
+    async fn get_message(
+        &mut self,
+        chatid: i64,
+        messageid: i32,
+    ) -> Result<Option<Message>, Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let messages = db.collection::<Message>("messages");
 
@@ -204,7 +211,7 @@ pub trait CallDB {
         &mut self,
         chatid: i64,
         messageid: i32,
-    ) -> DbResult<Option<String>> {
+    ) -> Result<Option<String>, Box<dyn std::error::Error>> {
         let msg = self.get_message(chatid, messageid).await?;
         Ok(msg.map(|m| m.token))
     }
@@ -214,7 +221,7 @@ pub trait CallDB {
         chatid: i64,
         messageid: i32,
         literal: &str,
-    ) -> DbResult<()> {
+    ) -> Result<(), Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let messages = db.collection::<Message>("messages");
 
@@ -234,7 +241,10 @@ pub trait CallDB {
         Ok(())
     }
 
-    async fn get_literal(&mut self, literal: &str) -> DbResult<Option<Literal>> {
+    async fn get_literal(
+        &mut self,
+        literal: &str,
+    ) -> Result<Option<Literal>, Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let messages = db.collection::<Literal>("literals");
 
@@ -243,13 +253,20 @@ pub trait CallDB {
         Ok(literal)
     }
 
-    async fn get_literal_value(&mut self, literal: &str) -> DbResult<Option<String>> {
+    async fn get_literal_value(
+        &mut self,
+        literal: &str,
+    ) -> Result<Option<String>, Box<dyn std::error::Error>> {
         let literal = self.get_literal(literal).await?;
 
         Ok(literal.map(|l| l.value))
     }
 
-    async fn set_literal(&mut self, literal: &str, valuestr: &str) -> DbResult<()> {
+    async fn set_literal(
+        &mut self,
+        literal: &str,
+        valuestr: &str,
+    ) -> Result<(), Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let literals = db.collection::<Literal>("literals");
 
@@ -264,14 +281,23 @@ pub trait CallDB {
         Ok(())
     }
 
-    async fn get_all_events(&mut self) -> DbResult<Vec<Event>> {
+    async fn get_all_events(&mut self) -> Vec<Event> {
         let db = self.get_database().await;
         let events = db.collection::<Event>("events");
 
-        events.find(doc! {}).await?.try_collect().await
+        events
+            .find(doc! {})
+            .await
+            .unwrap()
+            .map(|e| e.unwrap())
+            .collect()
+            .await
     }
 
-    async fn create_event(&mut self, event_datetime: chrono::DateTime<Utc>) -> DbResult<Event> {
+    async fn create_event(
+        &mut self,
+        event_datetime: chrono::DateTime<Utc>,
+    ) -> Result<Event, Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let events = db.collection::<Event>("events");
 
@@ -285,7 +311,7 @@ pub trait CallDB {
         Ok(new_event)
     }
 
-    async fn get_media(&mut self, literal: &str) -> DbResult<Vec<Media>> {
+    async fn get_media(&mut self, literal: &str) -> Result<Vec<Media>, Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let media = db.collection::<Media>("media");
 
@@ -298,7 +324,10 @@ pub trait CallDB {
         Ok(media_items)
     }
 
-    async fn is_media_group_exists(&mut self, media_group: &str) -> DbResult<bool> {
+    async fn is_media_group_exists(
+        &mut self,
+        media_group: &str,
+    ) -> Result<bool, Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let media = db.collection::<Media>("media");
 
@@ -310,7 +339,7 @@ pub trait CallDB {
         Ok(is_exists)
     }
 
-    async fn drop_media(&mut self, literal: &str) -> DbResult<usize> {
+    async fn drop_media(&mut self, literal: &str) -> Result<usize, Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let media = db.collection::<Media>("media");
 
@@ -322,7 +351,11 @@ pub trait CallDB {
         Ok(deleted_count as usize)
     }
 
-    async fn drop_media_except(&mut self, literal: &str, except_group: &str) -> DbResult<usize> {
+    async fn drop_media_except(
+        &mut self,
+        literal: &str,
+        except_group: &str,
+    ) -> Result<usize, Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let media = db.collection::<Media>("media");
 
@@ -343,7 +376,7 @@ pub trait CallDB {
         mediatype: &str,
         fileid: &str,
         media_group: Option<&str>,
-    ) -> DbResult<Media> {
+    ) -> Result<Media, Box<dyn std::error::Error>> {
         let db = self.get_database().await;
         let media = db.collection::<Media>("media");
 

src/db/tests.rs

@@ -1,5 +1,3 @@
-#![allow(clippy::unwrap_used)]
-
 use dotenvy;
 
 use super::CallDB;
@@ -8,20 +6,21 @@ use super::DB;
 async fn setup_db() -> DB {
     dotenvy::dotenv().unwrap();
     let db_url = std::env::var("DATABASE_URL").unwrap();
+    let db = DB::new(db_url).await;
 
-    DB::new(db_url).await.unwrap()
+    db
 }
 
 #[tokio::test]
 async fn test_get_media() {
     let mut db = setup_db().await;
 
-    let _result = db.drop_media("test_get_media_literal").await.unwrap();
+    let result = db.drop_media("test_get_media_literal").await.unwrap();
 
     let media_items = db.get_media("test_get_media_literal").await.unwrap();
     assert_eq!(media_items.len(), 0);
 
-    let _result = db
+    let result = db
         .add_media("test_get_media_literal", "photo", "file_id_1", None)
         .await
         .unwrap();
@@ -29,7 +28,7 @@ async fn test_get_media() {
     let media_items = db.get_media("test_get_media_literal").await.unwrap();
     assert_eq!(media_items.len(), 1);
 
-    let _result = db
+    let result = db
         .add_media("test_get_media_literal", "video", "file_id_2", None)
         .await
         .unwrap();
@@ -38,7 +37,7 @@ async fn test_get_media() {
     assert_eq!(media_items.len(), 2);
 
     // Clean up after test
-    let _result = db.drop_media("test_get_media_literal").await.unwrap();
+    let result = db.drop_media("test_get_media_literal").await.unwrap();
 }
 
 #[tokio::test]
@@ -49,9 +48,9 @@ async fn test_add_media() {
     let media_type = "photo";
     let file_id = "LjaldhAOh";
 
-    let _result = db.drop_media(literal).await.unwrap();
+    let result = db.drop_media(literal).await.unwrap();
 
-    let _result = db
+    let result = db
         .add_media(literal, media_type, file_id, None)
         .await
         .unwrap();
@@ -64,14 +63,14 @@ async fn test_add_media() {
     assert_eq!(media_items[0].file_id, file_id);
 
     // Clean up after test
-    let _result = db.drop_media(literal).await.unwrap();
+    let result = db.drop_media(literal).await.unwrap();
 }
 
 #[tokio::test]
 async fn test_drop_media() {
     let mut db = setup_db().await;
 
-    let _result = db
+    let result = db
         .add_media("test_drop_media_literal", "photo", "file_id_1", None)
         .await
         .unwrap();
@@ -80,14 +79,14 @@ async fn test_drop_media() {
     let media_items = db.get_media("test_drop_media_literal").await.unwrap();
     assert_eq!(media_items.len(), 1);
 
-    let _result = db.drop_media("test_drop_media_literal").await.unwrap();
+    let result = db.drop_media("test_drop_media_literal").await.unwrap();
 
     // Verify that the media has been dropped
     let media_items = db.get_media("test_drop_media_literal").await.unwrap();
     assert_eq!(media_items.len(), 0);
 
     // Clean up after test
-    let _result = db.drop_media("test_drop_media_literal").await.unwrap();
+    let result = db.drop_media("test_drop_media_literal").await.unwrap();
 }
 
 #[tokio::test]
@@ -136,7 +135,7 @@ async fn test_drop_media_except() {
     let media_items = db.get_media(literal).await.unwrap();
     assert_eq!(media_items.len(), 2);
 
-    let _deleted_count = db.drop_media_except(literal, media_group).await.unwrap();
+    let deleted_count = db.drop_media_except(literal, media_group).await.unwrap();
     let media_items = db.get_media(literal).await.unwrap();
     assert_eq!(media_items.len(), 0);
 
@@ -153,7 +152,7 @@ async fn test_drop_media_except() {
         .await
         .unwrap();
 
-    let _deleted_count = db.drop_media_except(literal, media_group).await.unwrap();
+    let deleted_count = db.drop_media_except(literal, media_group).await.unwrap();
     let media_items = db.get_media(literal).await.unwrap();
     assert_eq!(media_items.len(), 1);
     let _ = db.drop_media(literal).await.unwrap();
@@ -167,7 +166,7 @@ async fn test_drop_media_except() {
         .await
         .unwrap();
 
-    let _deleted_count = db.drop_media_except(literal, media_group).await.unwrap();
+    let deleted_count = db.drop_media_except(literal, media_group).await.unwrap();
     let media_items = db.get_media(literal).await.unwrap();
     assert_eq!(media_items.len(), 2);
 

src/main.rs

@@ -2,7 +2,6 @@ pub mod admin;
 pub mod db;
 pub mod mongodb_storage;
 
-use log::info;
 use std::time::Duration;
 
 use crate::admin::{admin_command_handler, AdminCommands};
@@ -12,11 +11,10 @@ use crate::mongodb_storage::MongodbStorage;
 
 use chrono::{DateTime, Utc};
 use chrono_tz::Asia;
-use db::DbError;
 use envconfig::Envconfig;
 use serde::{Deserialize, Serialize};
 use teloxide::dispatching::dialogue::serializer::Json;
-use teloxide::dispatching::dialogue::{GetChatId, Serializer};
+use teloxide::dispatching::dialogue::GetChatId;
 use teloxide::types::{
     InlineKeyboardButton, InlineKeyboardMarkup, InputFile, InputMedia, MediaKind, MessageKind,
     ParseMode, ReplyMarkup,
@@ -54,7 +52,7 @@ trait LogMsg {
 
 impl LogMsg for <teloxide::Bot as teloxide::prelude::Requester>::SendMessage {
     fn log(self) -> Self {
-        info!("msg: {}", self.text);
+        println!("msg: {}", self.text);
         self
     }
 }
@@ -84,21 +82,6 @@ impl BotController {
     }
 }
 
-#[derive(thiserror::Error, Debug)]
-pub enum BotError {
-    DBError(#[from] DbError),
-    TeloxideError(#[from] teloxide::RequestError),
-    StorageError(#[from] mongodb_storage::MongodbStorageError<<Json as Serializer<State>>::Error>),
-}
-
-pub type BotResult<T> = Result<T, BotError>;
-
-impl std::fmt::Display for BotError {
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        write!(f, "{:?}", self)
-    }
-}
-
 #[tokio::main]
 async fn main() -> Result<(), Box<dyn std::error::Error>> {
     dotenvy::dotenv()?;
@@ -108,39 +91,33 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let state_mgr = MongodbStorage::open(config.db_url.clone().as_ref(), "gongbot", Json).await?;
 
     // TODO: delete this in production
-    // allow because values are hardcoded and if they will be unparsable
-    // we should panic anyway
-    #[allow(clippy::unwrap_used)]
-    let events: Vec<DateTime<Utc>> = ["2025-04-09T18:00:00+04:00", "2025-04-11T16:00:00+04:00"]
+    let events: Vec<DateTime<Utc>> = vec!["2025-04-09T18:00:00+04:00", "2025-04-11T16:00:00+04:00"]
         .iter()
         .map(|d| DateTime::parse_from_rfc3339(d).unwrap().into())
         .collect();
 
     for event in events {
         match bc.db.create_event(event).await {
-            Ok(e) => info!("Created event {}", e._id),
-            Err(err) => info!("Failed to create event, error: {}", err),
+            Ok(e) => println!("Created event {}", e._id),
+            Err(err) => println!("Failed to create event, error: {}", err),
         }
     }
     //
 
     let handler = dptree::entry()
         .inspect(|u: Update| {
-            info!("{u:#?}"); // Print the update to the console with inspect
+            eprintln!("{u:#?}"); // Print the update to the console with inspect
         })
         .branch(Update::filter_callback_query().endpoint(callback_handler))
         .branch(command_handler(config))
         .branch(
             Update::filter_message()
                 .filter_async(async |msg: Message, mut db: DB| {
-                    let tguser = match msg.from.clone() {
-                        Some(user) => user,
-                        None => return false, // do nothing, cause its not usecase of function
-                    };
+                    let tguser = msg.from.unwrap();
                     let user = db
                         .get_or_init_user(tguser.id.0 as i64, &tguser.first_name)
                         .await;
-                    user.map(|u| u.is_admin).unwrap_or(false)
+                    user.is_admin
                 })
                 .enter_dialogue::<Message, MongodbStorage<Json>, State>()
                 .branch(
@@ -171,7 +148,11 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
     Ok(())
 }
 
-async fn callback_handler(bot: Bot, mut db: DB, q: CallbackQuery) -> BotResult<()> {
+async fn callback_handler(
+    bot: Bot,
+    mut db: DB,
+    q: CallbackQuery,
+) -> Result<(), teloxide::RequestError> {
     bot.answer_callback_query(&q.id).await?;
 
     if let Some(ref data) = q.data {
@@ -179,7 +160,10 @@ async fn callback_handler(bot: Bot, mut db: DB, q: CallbackQuery) -> BotResult<(
             "more_info" => {
                 answer_message(
                     &bot,
-                    q.chat_id().map(|i| i.0).unwrap_or(q.from.id.0 as i64),
+                    q.chat_id()
+                        .clone()
+                        .map(|i| i.0)
+                        .unwrap_or(q.from.id.0 as i64),
                     &mut db,
                     "more_info",
                     None as Option<InlineKeyboardMarkup>,
@@ -198,12 +182,16 @@ async fn edit_msg_cmd_handler(
     mut db: DB,
     dialogue: BotDialogue,
     msg: Message,
-) -> BotResult<()> {
+) -> Result<(), teloxide::RequestError> {
     match msg.reply_to_message() {
         Some(replied) => {
             let msgid = replied.id;
             // look for message in db and set text
-            let literal = match db.get_message_literal(msg.chat.id.0, msgid.0).await? {
+            let literal = match db
+                .get_message_literal(msg.chat.id.0, msgid.0)
+                .await
+                .unwrap()
+            {
                 Some(l) => l,
                 None => {
                     bot.send_message(msg.chat.id, "No such message found to edit. Look if you replying bot's message and this message is supposed to be editable").await?;
@@ -218,7 +206,8 @@ async fn edit_msg_cmd_handler(
                     lang,
                     is_caption_set: false,
                 })
-                .await?;
+                .await
+                .unwrap();
             bot.send_message(
                 msg.chat.id,
                 "Ok, now you have to send message text (formatting supported)",
@@ -239,44 +228,45 @@ async fn edit_msg_handler(
     dialogue: BotDialogue,
     (literal, lang, is_caption_set): (String, String, bool),
     msg: Message,
-) -> BotResult<()> {
+) -> Result<(), teloxide::RequestError> {
     use teloxide::utils::render::Renderer;
 
     let chat_id = msg.chat.id;
-    info!("Type: {:#?}", msg.kind);
+    println!("Type: {:#?}", msg.kind);
     let msg = if let MessageKind::Common(msg) = msg.kind {
         msg
     } else {
-        info!("Not a Common, somehow");
+        println!("Not a Common, somehow");
         return Ok(());
     };
 
     match msg.media_kind {
         MediaKind::Text(text) => {
-            db.drop_media(&literal).await?;
+            db.drop_media(&literal).await.unwrap();
             if is_caption_set {
                 return Ok(());
             };
             let html_text = Renderer::new(&text.text, &text.entities).as_html();
-            db.set_literal(&literal, &html_text).await?;
+            db.set_literal(&literal, &html_text).await.unwrap();
             bot.send_message(chat_id, "Updated text of message!")
                 .await?;
-            dialogue.exit().await?;
+            dialogue.exit().await.unwrap();
         }
         MediaKind::Photo(photo) => {
             let group = photo.media_group_id;
             if let Some(group) = group.clone() {
-                db.drop_media_except(&literal, &group).await?;
+                db.drop_media_except(&literal, &group).await.unwrap();
             } else {
-                db.drop_media(&literal).await?;
+                db.drop_media(&literal).await.unwrap();
             }
             let file_id = photo.photo[0].file.id.clone();
             db.add_media(&literal, "photo", &file_id, group.as_deref())
-                .await?;
+                .await
+                .unwrap();
             match photo.caption {
                 Some(text) => {
                     let html_text = Renderer::new(&text, &photo.caption_entities).as_html();
-                    db.set_literal(&literal, &html_text).await?;
+                    db.set_literal(&literal, &html_text).await.unwrap();
                     bot.send_message(chat_id, "Updated photo caption!").await?;
                 }
                 None => {
@@ -285,9 +275,10 @@ async fn edit_msg_handler(
                     // set text empty
                     if !db
                         .is_media_group_exists(group.as_deref().unwrap_or(""))
-                        .await?
+                        .await
+                        .unwrap()
                     {
-                        db.set_literal(&literal, "").await?;
+                        db.set_literal(&literal, "").await.unwrap();
                         bot.send_message(chat_id, "Set photo without caption")
                             .await?;
                     };
@@ -305,7 +296,8 @@ async fn edit_msg_handler(
                     lang,
                     is_caption_set: true,
                 })
-                .await?;
+                .await
+                .unwrap();
             tokio::spawn(async move {
                 tokio::time::sleep(Duration::from_millis(200)).await;
                 dialogue.exit().await.unwrap_or(());
@@ -314,17 +306,18 @@ async fn edit_msg_handler(
         MediaKind::Video(video) => {
             let group = video.media_group_id;
             if let Some(group) = group.clone() {
-                db.drop_media_except(&literal, &group).await?;
+                db.drop_media_except(&literal, &group).await.unwrap();
             } else {
-                db.drop_media(&literal).await?;
+                db.drop_media(&literal).await.unwrap();
             }
             let file_id = video.video.file.id;
             db.add_media(&literal, "video", &file_id, group.as_deref())
-                .await?;
+                .await
+                .unwrap();
             match video.caption {
                 Some(text) => {
                     let html_text = Renderer::new(&text, &video.caption_entities).as_html();
-                    db.set_literal(&literal, &html_text).await?;
+                    db.set_literal(&literal, &html_text).await.unwrap();
                     bot.send_message(chat_id, "Updated video caption!").await?;
                 }
                 None => {
@@ -333,9 +326,10 @@ async fn edit_msg_handler(
                     // set text empty
                     if !db
                         .is_media_group_exists(group.as_deref().unwrap_or(""))
-                        .await?
+                        .await
+                        .unwrap()
                     {
-                        db.set_literal(&literal, "").await?;
+                        db.set_literal(&literal, "").await.unwrap();
                         bot.send_message(chat_id, "Set video without caption")
                             .await?;
                     };
@@ -353,7 +347,8 @@ async fn edit_msg_handler(
                     lang,
                     is_caption_set: true,
                 })
-                .await?;
+                .await
+                .unwrap();
             tokio::spawn(async move {
                 tokio::time::sleep(Duration::from_millis(200)).await;
                 dialogue.exit().await.unwrap_or(());
@@ -370,7 +365,12 @@ async fn edit_msg_handler(
 
 fn command_handler(
     config: Config,
-) -> Handler<'static, DependencyMap, BotResult<()>, teloxide::dispatching::DpHandlerDescription> {
+) -> Handler<
+    'static,
+    DependencyMap,
+    Result<(), teloxide::RequestError>,
+    teloxide::dispatching::DpHandlerDescription,
+> {
     Update::filter_message()
         .branch(
             dptree::entry()
@@ -386,14 +386,11 @@ fn command_handler(
         .branch(
             dptree::entry()
                 .filter_async(async |msg: Message, mut db: DB| {
-                    let tguser = match msg.from.clone() {
-                        Some(user) => user,
-                        None => return false, // do nothing, cause its not usecase of function
-                    };
+                    let tguser = msg.from.unwrap();
                     let user = db
                         .get_or_init_user(tguser.id.0 as i64, &tguser.first_name)
                         .await;
-                    user.map(|u| u.is_admin).unwrap_or(false)
+                    user.is_admin
                 })
                 .filter_command::<AdminCommands>()
                 .endpoint(admin_command_handler),
@@ -405,20 +402,14 @@ async fn user_command_handler(
     bot: Bot,
     msg: Message,
     cmd: UserCommands,
-) -> BotResult<()> {
-    let tguser = match msg.from.clone() {
-        Some(user) => user,
-        None => return Ok(()), // do nothing, cause its not usecase of function
-    };
+) -> Result<(), teloxide::RequestError> {
+    let tguser = msg.from.clone().unwrap();
     let user = db
         .get_or_init_user(tguser.id.0 as i64, &tguser.first_name)
-        .await?;
-    let user = update_user_tg(user, &tguser);
-    user.update_user(&mut db).await?;
-    info!(
-        "MSG: {}",
-        msg.html_text().unwrap_or("|EMPTY_MESSAGE|".into())
-    );
+        .await;
+    let user = update_user_tg(user, msg.from.as_ref().unwrap());
+    user.update_user(&mut db).await.unwrap();
+    println!("MSG: {}", msg.html_text().unwrap());
     match cmd {
         UserCommands::Start => {
             let mut db2 = db.clone();
@@ -427,10 +418,9 @@ async fn user_command_handler(
                 msg.chat.id.0,
                 &mut db,
                 "start",
-                Some(make_start_buttons(&mut db2).await?),
+                Some(make_start_buttons(&mut db2).await),
             )
-            .await?;
-            Ok(())
+            .await
         }
         UserCommands::Help => {
             bot.send_message(msg.chat.id, UserCommands::descriptions().to_string())
@@ -446,12 +436,13 @@ async fn answer_message<RM: Into<ReplyMarkup>>(
     db: &mut DB,
     literal: &str,
     keyboard: Option<RM>,
-) -> BotResult<()> {
+) -> Result<(), teloxide::RequestError> {
     let text = db
         .get_literal_value(literal)
-        .await?
+        .await
+        .unwrap()
         .unwrap_or("Please, set content of this message".into());
-    let media = db.get_media(literal).await?;
+    let media = db.get_media(&literal).await.unwrap();
     let (chat_id, msg_id) = match media.len() {
         // just a text
         0 => {
@@ -461,7 +452,7 @@ async fn answer_message<RM: Into<ReplyMarkup>>(
                 None => msg,
             };
             let msg = msg.parse_mode(teloxide::types::ParseMode::Html);
-            info!("ENTS: {:?}", msg.entities);
+            println!("ENTS: {:?}", msg.entities);
             let msg = msg.await?;
 
             (msg.chat.id.0, msg.id.0)
@@ -563,14 +554,16 @@ async fn answer_message<RM: Into<ReplyMarkup>>(
             (msg[0].chat.id.0, msg[0].id.0)
         }
     };
-    db.set_message_literal(chat_id, msg_id, literal).await?;
+    db.set_message_literal(chat_id, msg_id, literal)
+        .await
+        .unwrap();
     Ok(())
 }
 
-async fn make_start_buttons(db: &mut DB) -> BotResult<InlineKeyboardMarkup> {
+async fn make_start_buttons(db: &mut DB) -> InlineKeyboardMarkup {
     let mut buttons: Vec<Vec<InlineKeyboardButton>> = db
         .get_all_events()
-        .await?
+        .await
         .iter()
         .map(|e| {
             vec![InlineKeyboardButton::callback(
@@ -584,12 +577,12 @@ async fn make_start_buttons(db: &mut DB) -> BotResult<InlineKeyboardMarkup> {
         "more_info",
     )]);
 
-    Ok(InlineKeyboardMarkup::new(buttons))
+    InlineKeyboardMarkup::new(buttons)
 }
 
-async fn echo(bot: Bot, msg: Message) -> BotResult<()> {
+async fn echo(bot: Bot, msg: Message) -> Result<(), teloxide::RequestError> {
     if let Some(photo) = msg.photo() {
-        info!("File ID: {}", photo[0].file.id);
+        println!("File ID: {}", photo[0].file.id);
     }
     bot.send_message(msg.chat.id, msg.html_text().unwrap_or("UNWRAP".into()))
         .parse_mode(teloxide::types::ParseMode::Html)

src/mongodb_storage.rs

@@ -36,26 +36,6 @@ pub struct Dialogue {
     dialogue: Vec<u32>,
 }
 
-#[derive(Debug, thiserror::Error)]
-pub enum MongodbStorageError<SE>
-where
-    SE: Debug + Display,
-{
-    MongodbError(#[from] mongodb::error::Error),
-    SerdeError(SE),
-}
-
-pub type MongodbStorageResult<T, SE> = Result<T, MongodbStorageError<SE>>;
-
-impl<SE> std::fmt::Display for MongodbStorageError<SE>
-where
-    SE: Debug + Display,
-{
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        write!(f, "{:?}", self)
-    }
-}
-
 impl<S, D> Storage<D> for MongodbStorage<S>
 where
     S: Send + Sync + Serializer<D> + 'static,
@@ -63,7 +43,7 @@ where
 
     <S as Serializer<D>>::Error: Debug + Display,
 {
-    type Error = MongodbStorageError<<S as Serializer<D>>::Error>;
+    type Error = mongodb::error::Error;
 
     fn remove_dialogue(
         self: std::sync::Arc<Self>,
@@ -76,8 +56,7 @@ where
             let d = self.database.collection::<Dialogue>("dialogues");
             d.delete_one(doc! { "chat_id": chat_id.0 })
                 .await
-                .map(|_| ())?;
-            Ok(())
+                .map(|_| ())
         })
     }
 
@@ -97,14 +76,9 @@ where
                         },
                         doc! {
                             "$set": doc! {
-                            "dialogue": self.serializer.serialize(&dialogue)
-                                .map_err(MongodbStorageError::SerdeError)?
-                                .into_iter().map(|v| v as u32).collect::<Vec<u32>>()
+                                "dialogue": self.serializer.serialize(&dialogue).unwrap().into_iter().map(|v| v as u32).collect::<Vec<u32>>()
                             }
-                },
-            )
-            .upsert(true)
-            .await?;
+                    }).upsert(true).await?;
             Ok(())
         })
     }
@@ -115,13 +89,8 @@ where
     ) -> BoxFuture<'static, Result<Option<D>, Self::Error>> {
         Box::pin(async move {
             let d = self.database.collection::<Dialogue>("dialogues");
-            let d = d.find_one(doc! { "chat_id": chat_id.0 }).await?;
-            let d = match d {
-                Some(d) => d,
-                None => return Ok(None),
-            };
-            let d = self
-                .serializer
+            Ok(d.find_one(doc! { "chat_id": chat_id.0 }).await?.map(|d| {
+                self.serializer
                     .deserialize(
                         d.dialogue
                             .into_iter()
@@ -129,9 +98,8 @@ where
                             .collect::<Vec<_>>()
                             .as_slice(),
                     )
-                .map_err(MongodbStorageError::SerdeError)?;
-
-            Ok(Some(d))
+                    .unwrap()
+            }))
         })
     }
 }