Wrap Config object in Arc.

Author: smarnach

src/authorizations/users.rs

@@ -14,7 +14,6 @@ pub fn is_authorized(id: &str, config: &Config) -> bool {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use std::env;
 
     fn test_config() -> Config {
         Config::new(

src/commands/heroku.rs

@@ -16,9 +16,7 @@ struct HerokuApp {
 
 #[command]
 pub fn get_apps(ctx: &mut Context, msg: &Message, _args: Args) -> CommandResult {
-    let ctx_clone = ctx.clone();
-    let data = ctx_clone.data.read();
-    let config = data.get::<Config>().expect("Expected config");
+    let config = ctx.data.read().get::<Config>().expect("Expected config").clone();
 
     let response = heroku_client(&config.heroku_api_key)
         .get()

src/commands/ping.rs

@@ -9,11 +9,9 @@ use crate::config::config::Config;
 fn ping(ctx: &mut Context, msg: &Message) -> CommandResult {
     println!("running pong command");
 
-    let ctx_clone = ctx.clone();
-    let data = ctx_clone.data.read();
-    let config = data.get::<Config>().expect("Expected config");
+    let config = ctx.data.read().get::<Config>().expect("Expected config").clone();
 
-    if is_authorized(&msg.author.id.to_string(), config) {
+    if is_authorized(&msg.author.id.to_string(), &*config) {
         msg.reply(ctx, "Pong!")?;
     } else {
         msg.reply(

src/config.rs

@@ -1,4 +1,5 @@
 pub mod config {
+    use std::sync::Arc;
     use serenity::prelude::TypeMapKey;
 
     #[derive(Debug)]
@@ -23,6 +24,6 @@ pub mod config {
     }
 
     impl TypeMapKey for Config {
-        type Value = Config;
+        type Value = Arc<Config>;
     }
 }

src/lib.rs

@@ -1,3 +1,5 @@
+use std::sync::Arc;
+
 use serenity::client::Client;
 use serenity::framework::standard::{macros::group, StandardFramework};
 use serenity::model::gateway::Ready;
@@ -32,7 +34,7 @@ pub fn run(config: Config) {
     // that is passed to each of the commands
     {
         let mut data = client.data.write();
-        data.insert::<Config>(config);
+        data.insert::<Config>(Arc::new(config));
     }
 
     client.with_framework(