Don't include port if port is 80

src/cli.rs

@@ -2,7 +2,7 @@ use clap::ArgMatches;
 use std::fs;
 use std::io;
 
-use crate::client::{create_snippet, OutgoingSnippet, ServerConfig};
+use crate::client::{build_url_from_config, create_snippet, OutgoingSnippet, ServerConfig};
 
 fn get_snippet_from_matches(matches: &ArgMatches) -> io::Result<OutgoingSnippet> {
     let title = matches.value_of("name").unwrap();
@@ -38,13 +38,13 @@ fn get_server_config_from_matches(matches: &ArgMatches) -> ServerConfig {
 
 pub fn new_snippet(matches: &ArgMatches) -> io::Result<()> {
     let snippet = get_snippet_from_matches(matches)?;
-    println!("{:?}", snippet);
     let config = get_server_config_from_matches(matches);
+    let url = build_url_from_config(&config);
 
     match create_snippet(snippet, &config) {
         Ok(resp) => println!(
-            "Success! Your new snippet is available at {}://{}:{}/snippets/{}",
-            config.protocol, config.hostname, config.port, resp.id
+            "Success! Your new snippet is available at {}/snippets/{}",
+            url, resp.id
         ),
         Err(err) => println!("{}", err),
     }

src/client.rs

@@ -24,19 +24,23 @@ pub struct ServerConfig {
     pub port: i32,
 }
 
+pub fn build_url_from_config(config: &ServerConfig) -> String {
+    match config.port {
+        80 => format!("{}://{}", config.protocol, config.hostname),
+        _ => format!("{}://{}:{}", config.protocol, config.hostname, config.port),
+    }
+}
+
 pub fn create_snippet(
     snippet: OutgoingSnippet,
     config: &ServerConfig,
 ) -> Result<IncomingSnippet, Box<std::error::Error>> {
     let client = reqwest::Client::new();
+    let url = build_url_from_config(config);
     let resp: IncomingSnippet = client
-        .post(&format!(
-            "{}://{}:{}/api/snippets",
-            config.protocol, config.hostname, config.port,
-        ))
+        .post(&format!("{}/api/snippets", url))
         .json(&snippet)
         .send()?
         .json()?;
-
     Ok(resp)
 }