client/server - update message - update authentication

client/app/pages/home/home.tsx

@@ -6,7 +6,6 @@ import './home.scss';
 
 import header_forms from '../../assets/header-forums.gif';
 import support from '../../assets/support.gif';
-// import serverstatus from '../../assets/serverstatus.gif';
 import uicustomizations from '../../assets/uicustomizations.gif';
 import bugs from '../../assets/bugs.gif';
 import realms from '../../assets/realms.gif';
@@ -94,9 +93,10 @@ export class Home extends React.Component<Props, State> {
             <b>Welcome to the unofficial Classic WoW Forums!</b>
           </div>
           <p>
-            This site is made with the intention of providing a web forum
-            that replicates the World of Warcraft forums as they were
-            back in 2005.
+            I created this site with the intention of providing
+            a fully functional and accurate representation of
+            the World of Warcraft forums as they were
+            back in 2005. <Link to={'/t/137/1'}>Please read this first!</Link>
           </p>
 
           <p>

client/app/scss/style.scss

@@ -172,7 +172,7 @@ div {
 }
 
 .blue-post {
-  p {
+  p, li, {
     color: $bluePost;
   }
 }

lib/myapp/auth/token.ex

@@ -1,15 +1,18 @@
 defmodule MyApp.Guardian.Auth.Token do
   alias MyApp.Guardian
 
+  # one month in seconds
+  defp one_month(), do: 2592000
+
   @spec add_token_and_map_claims(map | {atom, any}) :: {:ok, map} | {:error, String.t}
   def add_token_and_map_claims(user) when is_map(user) do
 
     claims = user
-    |> Map.take([:id, :battletag, :battle_net_id, "access_token"]) # take values from user object to map to claims
+    |> Map.take([:id, :battletag, :battle_net_id, :access_token]) # take values from user object to map to claims
     |> Guardian.add_permissions(get_permissions(user))
 
     # set token expiration to the same as the battlenet token
-    case Guardian.encode_and_sign(user, claims, ttl: {user["expires_in"], :seconds}) do
+    case Guardian.encode_and_sign(user, claims, ttl: {Map.get(user, :expires_in) || one_month(), :seconds}) do
       {:ok, token, _claims} -> {:ok, Map.merge(user, %{token: token})}
       {:error, error} -> {:error, error}
     end

lib/myapp/data/user.ex

@@ -82,7 +82,7 @@ defmodule MyApp.Data.User do
   # need to add token back to map because we don't store it in the database
   defp add_extra_params({:error, error}, _), do: {:error, error}
   defp add_extra_params({:ok, user}, params) do
-    {:ok, Map.merge(user, params)}
+    {:ok, Map.merge(user, %{access_token: params["access_token"], expires_in: params["expires_in"]})}
   end
 
   defp insert_battlenet_user(params) do

test/myapp/data/user_test.exs

@@ -28,6 +28,7 @@ defmodule MyApp.Data.UserTest do
       battletag: "mgerb",
       id: user.id,
       permissions: "user",
+      expires_in: nil,
     }
 
     # call upsert again with same battletag
@@ -41,6 +42,7 @@ defmodule MyApp.Data.UserTest do
       battletag: "mgerb",
       id: user.id,
       permissions: "user",
+      expires_in: nil,
       character_avatar: nil,
       character_class: nil,
       character_guild: nil,

test/myapp_web/controllers/reply_controller_test.exs

@@ -63,7 +63,7 @@ defmodule MyAppWeb.ReplyControllerTest do
 
     # new reply should work
     conn = post(new_conn, "/api/reply", %{"content" => "c", "thread_id" => thread_id})
-    body = conn |> response(200) |> Poison.decode!
+    conn |> response(200) |> Poison.decode!
   end
 
   test "user update reply" do