From a8aeb40b957b9fa22aaf53070d29abe67a401232 Mon Sep 17 00:00:00 2001
From: Trinity Pointard <trinity.pointard@insa-rennes.fr>
Date: Tue, 22 May 2018 17:35:16 +0200
Subject: [PATCH] add trailing slashes to links where they were missing

Signed-off-by: Trinity Pointard <trinity.pointard@insa-rennes.fr>
---
 src/models/posts.rs                | 2 +-
 src/routes/blogs.rs                | 2 +-
 src/routes/posts.rs                | 2 +-
 src/routes/user.rs                 | 6 +++---
 templates/notifications/index.tera | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/models/posts.rs b/src/models/posts.rs
index dcb94837..35f08b30 100644
--- a/src/models/posts.rs
+++ b/src/models/posts.rs
@@ -183,7 +183,7 @@ impl IntoId for Post {
 
 impl Object for Post {
     fn compute_id(&self, conn: &PgConnection) -> String {
-        ap_url(format!("{}/~/{}/{}", BASE_URL.as_str(), self.get_blog(conn).actor_id, self.slug))
+        ap_url(format!("{}/~/{}/{}/", BASE_URL.as_str(), self.get_blog(conn).actor_id, self.slug))
     }
 
     fn serialize(&self, conn: &PgConnection) -> serde_json::Value {
diff --git a/src/routes/blogs.rs b/src/routes/blogs.rs
index 08fea83b..2df26cf5 100644
--- a/src/routes/blogs.rs
+++ b/src/routes/blogs.rs
@@ -77,7 +77,7 @@ fn create(conn: DbConn, data: Form<NewBlogForm>, user: User) -> Redirect {
         is_owner: true
     });
     
-    Redirect::to(format!("/~/{}", slug).as_str())
+    Redirect::to(format!("/~/{}/", slug).as_str())
 }
 
 #[get("/~/<name>/outbox")]
diff --git a/src/routes/posts.rs b/src/routes/posts.rs
index e213c869..61a917f4 100644
--- a/src/routes/posts.rs
+++ b/src/routes/posts.rs
@@ -96,5 +96,5 @@ fn create(blog_name: String, data: Form<NewPostForm>, user: User, conn: DbConn)
     let act = post.create_activity(&*conn);
     broadcast(&*conn, &user, act, user.get_followers(&*conn));
 
-    Redirect::to(format!("/~/{}/{}", blog_name, slug).as_str())
+    Redirect::to(format!("/~/{}/{}/", blog_name, slug).as_str())
 }
diff --git a/src/routes/user.rs b/src/routes/user.rs
index 807168a8..6f4bb615 100644
--- a/src/routes/user.rs
+++ b/src/routes/user.rs
@@ -21,7 +21,7 @@ use models::{
 
 #[get("/me")]
 fn me(user: User) -> Redirect {
-    Redirect::to(format!("/@/{}", user.username).as_ref())
+    Redirect::to(format!("/@/{}/", user.username).as_ref())
 }
 
 #[get("/@/<name>", rank = 2)]
@@ -64,7 +64,7 @@ fn follow(name: String, conn: DbConn, user: User) -> Redirect {
     act.set_object_object(user.into_activity(&*conn)).unwrap();
     act.object_props.set_id_string(format!("{}/follow/{}", user.ap_url, target.ap_url)).unwrap();
     broadcast(&*conn, &user, act, vec![target]);
-    Redirect::to(format!("/@/{}", name).as_ref())
+    Redirect::to(format!("/@/{}/", name).as_ref())
 }
 
 #[get("/@/<name>/followers", rank = 2)]
@@ -155,7 +155,7 @@ fn create(conn: DbConn, data: Form<NewUserForm>) -> Result<Redirect, String> {
             User::hash_pass(form.password.to_string()),
             inst.id
         )).update_boxes(&*conn);
-        Ok(Redirect::to(format!("/@/{}", data.get().username).as_str()))        
+        Ok(Redirect::to(format!("/@/{}/", data.get().username).as_str()))
     } else {
         Err(String::from("Passwords don't match"))
     }
diff --git a/templates/notifications/index.tera b/templates/notifications/index.tera
index 3af0de19..d25dedb0 100644
--- a/templates/notifications/index.tera
+++ b/templates/notifications/index.tera
@@ -9,7 +9,7 @@ Notifications
     <div class="list">
         {% for notification in notifications %}
             <div class="card">
-                <h3><a href="{% if notification.link %}{{ notification.link }}{% else %}#{% endif %}">{{ notification.title }}</h3>
+                <h3><a href="{% if notification.link %}{{ notification.link }}/{% else %}#{% endif %}">{{ notification.title }}</h3>
                 {% if notification.content %}
                     <p>{{ notification.content }}</p>
                 {% endif %}