From db3ef7640a9524ca2cff23b83ed5b7aab5881534 Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 06:41:27 -0400
Subject: [PATCH 01/10] Renamed a variable

sorry it was bothering me.
---
 client/src/views/Login.vue | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/client/src/views/Login.vue b/client/src/views/Login.vue
index a50bf9d..84d09b7 100644
--- a/client/src/views/Login.vue
+++ b/client/src/views/Login.vue
@@ -3,38 +3,38 @@ import { ref, computed } from 'vue'
 import { state } from '../state';
 import router from '../router'
 
-const name = ref("")
-const password = ref("")
+const $name = ref("")
+const $password = ref("")
 const signUpWait = ref(false)
 const loginWait = ref(false)
 const signUpClass = computed(() => `submit button is-success ${signUpWait.value ? 'is-loading' : ''}`)
 const loginClass = computed(() => `submit button is-info ${loginWait.value ? 'is-loading' : ''}`)
 
 async function signUp() {
-    const $name = name.value
+    const name = $name.value, password = $password.value
     signUpWait.value = true
     const result = await fetch("/api/v1/auth", {
         method: 'POST',
-        body: JSON.stringify({ name: $name, password: password.value }),
+        body: JSON.stringify({ name, password }),
         headers: {'Content-Type': 'application/json'}
     })
     if (result.ok) {
-        state.user = { name: $name }
+        state.user = { name }
         await state.repopulate()
         router.push("/")
     }
 }
 
 async function login() {
-    const $name = name.value
+    const name = $name.value, password = $password.value
     loginWait.value = true
     const result = await fetch("/api/v1/auth", {
         method: 'PUT',
-        body: JSON.stringify({ name: $name, password: password.value }),
+        body: JSON.stringify({ name, password }),
         headers: {'Content-Type': 'application/json'}
     })
     if (result.ok) {
-        state.user = { name: $name }
+        state.user = { name }
         await state.repopulate()
         router.push("/")
     }
@@ -54,14 +54,14 @@ if(state.user?.name) router.push("/")
                 <div class="field">
                     <label for="username" class=label>Name</label>
                     <div class="control">
-                        <input type="text" name="username" class="input" v-model="name" />
+                        <input type="text" name="username" class="input" v-model="$name" />
                     </div>
                 </div>
 
                 <div class="field">
                     <label for="password" class="label">Password</label>
                     <div class="control">
-                        <input type="password" name="password" class="input" v-model="password" />
+                        <input type="password" name="password" class="input" v-model="$password" />
                     </div>
                 </div>
             </section>

From 5aa16762f7104bdbdc251d3c102381a5a7120870 Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 06:44:17 -0400
Subject: [PATCH 02/10] Quick tweak

---
 Makefile | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Makefile b/Makefile
index bc6334b..f97671d 100644
--- a/Makefile
+++ b/Makefile
@@ -7,6 +7,7 @@ client/dist/index.html:
 build-client:	client/dist/index.html
 
 start-server:	build-client
+	-mkdir db.mount
 	docker compose up --build -d
 
 clean:

From 003383e45565b5463fa73c48e7897aec2ecb0c70 Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 06:54:42 -0400
Subject: [PATCH 03/10] Fixed logout flow

---
 client/src/components/NavBar.vue | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/client/src/components/NavBar.vue b/client/src/components/NavBar.vue
index 64a6d6e..9b3699d 100644
--- a/client/src/components/NavBar.vue
+++ b/client/src/components/NavBar.vue
@@ -2,11 +2,12 @@
 import { RouterLink } from 'vue-router';
 import { error } from '../error'
 import router from "../router";
+import { state } from '../state'
 
 async function logOut() {
     const result = await fetch('/api/v1/auth', {method: 'DELETE'})
     if(!result.ok) return error('failed to log out')
-    console.debug('logged out')
+    state.user = undefined
     router.push('/login')
 }
 </script>

From dafdd491f959741709e05795594ca70ef863c387 Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 07:03:16 -0400
Subject: [PATCH 04/10] Move logOut function into method on state

---
 client/src/components/NavBar.vue | 9 +--------
 client/src/state.ts              | 8 ++++++++
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/client/src/components/NavBar.vue b/client/src/components/NavBar.vue
index 9b3699d..9a09c57 100644
--- a/client/src/components/NavBar.vue
+++ b/client/src/components/NavBar.vue
@@ -1,15 +1,8 @@
 <script setup lang="ts">
 import { RouterLink } from 'vue-router';
 import { error } from '../error'
-import router from "../router";
 import { state } from '../state'
 
-async function logOut() {
-    const result = await fetch('/api/v1/auth', {method: 'DELETE'})
-    if(!result.ok) return error('failed to log out')
-    state.user = undefined
-    router.push('/login')
-}
 </script>
 <template>
     <nav class="navbar" role="navigation" aria-label="main navigation">
@@ -32,7 +25,7 @@ async function logOut() {
                 </RouterLink>
             </div>
             <div class="navbar-item">
-                <button class="button is-info" @click="logOut">
+                <button class="button is-info" @click="state.logOut()">
                     Log Out
                 </button>
             </div>
diff --git a/client/src/state.ts b/client/src/state.ts
index 9e08abb..b0e94fa 100644
--- a/client/src/state.ts
+++ b/client/src/state.ts
@@ -3,6 +3,7 @@ import { Track } from "./track"
 import { Tick } from './ticks'
 import { error } from "./error"
 import { getCookie } from "./util";
+import router from './router'
 
 enum State {
     Unfetched,
@@ -130,6 +131,13 @@ class AppState {
         const response = await fetch(`/api/v1/tracks/${trackID}`, { method: "DELETE" })
         if (!response.ok) error(`error deleting track with ID ${trackID}: ${response.statusText} (${response.status})`)
     }
+    async logOut() {
+        const result = await fetch('/api/v1/auth', {method: 'DELETE'})
+        if(!result.ok) return error('failed to log out')
+        this.user = undefined
+        router.push('/login')
+    }
 }
 
+
 export const state = reactive(new AppState)

From cd16208dd777f2a002f6a66b472c6050268c03ad Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 11:01:14 -0400
Subject: [PATCH 05/10] Move dateQuery to util.ts

---
 client/src/state.ts | 11 +----------
 client/src/util.ts  | 11 ++++++++++-
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/client/src/state.ts b/client/src/state.ts
index b0e94fa..3f13eb9 100644
--- a/client/src/state.ts
+++ b/client/src/state.ts
@@ -2,7 +2,7 @@ import { reactive } from "vue"
 import { Track } from "./track"
 import { Tick } from './ticks'
 import { error } from "./error"
-import { getCookie } from "./util";
+import { getCookie, dateQuery } from "./util";
 import router from './router'
 
 enum State {
@@ -11,15 +11,6 @@ enum State {
     Fetched,
 }
 
-function dateQuery(date: Date): URLSearchParams {
-    let query = new URLSearchParams()
-    query.set("year", date.getUTCFullYear().toString())
-    query.set("month", (date.getUTCMonth() + 1).toString())
-    // good thing I still had this ^^^^^^^^^^^^^^ in mind when I wrote this 😬
-    query.set("day", date.getUTCDate().toString())
-    return query
-}
-
 interface LoggedInUser {
     name: string
 }
diff --git a/client/src/util.ts b/client/src/util.ts
index ebbda1c..023079c 100644
--- a/client/src/util.ts
+++ b/client/src/util.ts
@@ -5,4 +5,13 @@ export function getCookie(key: string): string | null {
     if(end === -1)
         end = undefined
     return document.cookie.substring(start + key.length + 1, end)
-}
\ No newline at end of file
+}
+
+export function dateQuery(date: Date): URLSearchParams {
+    let query = new URLSearchParams()
+    query.set("year", date.getUTCFullYear().toString())
+    query.set("month", (date.getUTCMonth() + 1).toString())
+    // good thing I still had this ^^^^^^^^^^^^^^ in mind when I wrote this 😬
+    query.set("day", date.getUTCDate().toString())
+    return query
+}

From 89b018098985ee5ab51748357437822bebe0beb7 Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 11:05:15 -0400
Subject: [PATCH 06/10] Move API calls from state to Track methods

---
 client/src/components/NavBar.vue        |  1 -
 client/src/components/TickComponent.vue |  4 +-
 client/src/components/TrackIcon.vue     |  4 +-
 client/src/state.ts                     | 32 +------------
 client/src/track.ts                     | 61 +++++++++++++++++++++++++
 client/src/views/NewTrackView.vue       |  3 +-
 6 files changed, 67 insertions(+), 38 deletions(-)

diff --git a/client/src/components/NavBar.vue b/client/src/components/NavBar.vue
index 9a09c57..145a817 100644
--- a/client/src/components/NavBar.vue
+++ b/client/src/components/NavBar.vue
@@ -1,6 +1,5 @@
 <script setup lang="ts">
 import { RouterLink } from 'vue-router';
-import { error } from '../error'
 import { state } from '../state'
 
 </script>
diff --git a/client/src/components/TickComponent.vue b/client/src/components/TickComponent.vue
index 8b1d863..47d57e0 100644
--- a/client/src/components/TickComponent.vue
+++ b/client/src/components/TickComponent.vue
@@ -15,8 +15,8 @@ const className = computed(() => isSet.value ? "button is-rounded is-info" : "bu
 
 async function toggle() {
     if (isSet.value) {
-        await state.taskMarkedIncomplete(props.track, props.date)
+        await props.track.markIncomplete(props.date)
     } else
-        await state.taskCompleted(props.track, props.date)
+        await props.track.markComplete(props.date)
 }
 </script>
\ No newline at end of file
diff --git a/client/src/components/TrackIcon.vue b/client/src/components/TrackIcon.vue
index 273987e..e5aeabf 100644
--- a/client/src/components/TrackIcon.vue
+++ b/client/src/components/TrackIcon.vue
@@ -1,12 +1,12 @@
 <script setup lang="ts">
-import { state } from '../state';
+import { Track } from '../track';
 
 const props = defineProps<{icon: String, id: number|undefined}>()
 
 const del = () => {
     if(props.id)
         if(confirm("are you sure you want to delete this track?"))
-            state.removeTrack(props.id)
+            Track.deleteById(props.id)
 }
 </script>
 <template>
diff --git a/client/src/state.ts b/client/src/state.ts
index 3f13eb9..028ae4b 100644
--- a/client/src/state.ts
+++ b/client/src/state.ts
@@ -2,7 +2,7 @@ import { reactive } from "vue"
 import { Track } from "./track"
 import { Tick } from './ticks'
 import { error } from "./error"
-import { getCookie, dateQuery } from "./util";
+import { getCookie } from "./util";
 import router from './router'
 
 enum State {
@@ -92,36 +92,6 @@ class AppState {
         if (this.state != State.Unfetched) return
         await this.repopulate()
     }
-    async taskCompleted(track: Track, date: Date): Promise<Tick> {
-        const query = dateQuery(date)
-        const response: Response = await fetch(`/api/v1/tracks/${track.id}/ticked?${query.toString()}`, { method: "PATCH" })
-        const body = await response.text()
-        if (!response.ok) {
-            error(body)
-            throw new Error(`error setting tick for track ${track.id} ("${track.name}"): ${response.status} ${response.statusText}`)
-        }
-        return JSON.parse(body)
-    }
-    async taskMarkedIncomplete(track: Track, date: Date) {
-        const query = dateQuery(date)
-        const { ok, status, statusText } = await fetch(`/api/v1/tracks/${track.id}/all-ticks?${query.toString()}`, { method: 'DELETE' })
-        if (!ok)
-            error(`error deleting ticks for ${track.id}: ${statusText} (${status})`)
-    }
-    async addTrack(track: Track): Promise<boolean> {
-        const response = await fetch('/api/v1/tracks', {
-            method: "POST",
-            body: JSON.stringify(track),
-            headers: { "Content-Type": "application/json" }
-        })
-        if (!response.ok)
-            error(`error submitting track: ${track}: ${response.statusText} (${response.status})`)
-        return response.ok
-    }
-    async removeTrack(trackID: number) {
-        const response = await fetch(`/api/v1/tracks/${trackID}`, { method: "DELETE" })
-        if (!response.ok) error(`error deleting track with ID ${trackID}: ${response.statusText} (${response.status})`)
-    }
     async logOut() {
         const result = await fetch('/api/v1/auth', {method: 'DELETE'})
         if(!result.ok) return error('failed to log out')
diff --git a/client/src/track.ts b/client/src/track.ts
index 9893eca..f7525ad 100644
--- a/client/src/track.ts
+++ b/client/src/track.ts
@@ -1,5 +1,6 @@
 import { error } from "./error"
 import { Tick, ITick } from './ticks'
+import { dateQuery } from "./util"
 
 export interface ITrack {
     id?: number
@@ -48,6 +49,34 @@ export class Track implements ITrack {
         this.fetchTicks = this.fetchTicks.bind(this)
     }
 
+    /**
+     * Add this track to the database. A `TrackAdded` event should have been
+     * received from the server on the event stream by the time this returns.
+     * 
+     * @returns whether or not the query succeeded
+     */
+    async create(): Promise<boolean> {
+        // note that this.id is expected to be `undefined` here.
+        const response = await fetch('/api/v1/tracks', {
+            method: "POST",
+            body: JSON.stringify(this),
+            headers: { "Content-Type": "application/json" }
+        })
+        if (!response.ok)
+            error(`error submitting track ${this.name}: ${response.statusText} (${response.status})`)
+        return response.ok
+    }
+
+    async delete() {
+        const id = this.id
+        if (id) await Track.deleteById(id)
+    }
+
+    static async deleteById(id: number) {
+        const response = await fetch(`/api/v1/tracks/${id}`, { method: "DELETE" })
+        if (!response.ok) error(`error deleting track with ID ${id}: ${response.statusText} (${response.status})`)
+    }
+
     static fromJSON(track: ITrack): Track {
         return new Track(track.id, track.name, track.description, track.icon, track.enabled, track.multiple_entries_per_day, track.color, track.order)
     }
@@ -98,4 +127,36 @@ export class Track implements ITrack {
         }
         return []
     }
+    /**
+     * Mark this track as being completed on the given date. A `TickAdded` event
+     * should have been received from the server on the event stream by the time
+     * this returns.
+     * 
+     * @param date the date the task was completed
+     * @returns the decoded server API response
+     */
+    async markComplete(date: Date) {
+        const query = dateQuery(date)
+        const response: Response = await fetch(`/api/v1/tracks/${this.id}/ticked?${query.toString()}`, { method: "PATCH" })
+        const body = await response.text()
+        if (!response.ok) {
+            error(body)
+            throw new Error(`error setting tick for track ${this.id} ("${this.name}"): ${response.status} ${response.statusText}`)
+        }
+        return JSON.parse(body)
+    }
+    /**
+     * Mark this track as being incomplete on the given date. A `TickAdded` event
+     * should have been received from the server on the event stream by the time
+     * this returns.
+     * 
+     * @param date the date the task was completed
+     * @returns the decoded server API response
+     */
+    async markIncomplete(date: Date) {
+        const query = dateQuery(date)
+        const { ok, status, statusText } = await fetch(`/api/v1/tracks/${this.id}/all-ticks?${query.toString()}`, { method: 'DELETE' })
+        if (!ok)
+            error(`error deleting ticks for ${this.id}: ${statusText} (${status})`)
+    }
 }
diff --git a/client/src/views/NewTrackView.vue b/client/src/views/NewTrackView.vue
index 8b2ed62..a15e26a 100644
--- a/client/src/views/NewTrackView.vue
+++ b/client/src/views/NewTrackView.vue
@@ -2,7 +2,6 @@
 import { RouterLink, useRouter } from 'vue-router';
 import { Track } from '../track';
 import { computed, ref } from 'vue';
-import { state } from '../state';
 
 const props = defineProps<{ initialState?: Track }>()
 const router = useRouter()
@@ -26,7 +25,7 @@ const submit = async () => {
     const track = new Track(undefined, name.value, description.value,
         icon.value, Number(enabled.value), Number(multipleEntriesPerDay.value),
         color.value, order.value)
-    if (await state.addTrack(track))
+    if (await track.create())
         router.push('/')
 }
 </script>

From 424bc15512460864637282890ca8ed014302ef3f Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 11:18:23 -0400
Subject: [PATCH 07/10] prod needs the cookie secret too, arguably more than
 the others!

---
 docker-compose_prod.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docker-compose_prod.yml b/docker-compose_prod.yml
index 71405e1..09483f7 100644
--- a/docker-compose_prod.yml
+++ b/docker-compose_prod.yml
@@ -14,7 +14,7 @@ services:
       POSTGRES_USER: kalkutago
       POSTGRES_DB: kalkutago
       POSTGRES_HOST: database
-    secrets: [ postgres-password ]
+    secrets: [ postgres-password, cookie-secret ]
     depends_on: [ database ]
     volumes:
       - ./client/dist:/src/public:ro

From bc6f06e210c3dce810b61879deb8d4adb96def78 Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 11:21:36 -0400
Subject: [PATCH 08/10] DEFAULT_COST + 2 is 2 much

---
 server/src/entities/users.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server/src/entities/users.rs b/server/src/entities/users.rs
index 5805910..57e0d6a 100644
--- a/server/src/entities/users.rs
+++ b/server/src/entities/users.rs
@@ -53,7 +53,7 @@ impl ActiveModel {
     pub fn new(name: impl AsRef<str>, password: impl AsRef<str>) -> error::Result<Self> {
         use sea_orm::ActiveValue::Set;
         let name = Set(name.as_ref().to_string());
-        let password_hash = Set(hash(password.as_ref(), DEFAULT_COST + 2)?);
+        let password_hash = Set(hash(password.as_ref(), DEFAULT_COST)?);
         Ok(Self {
             name,
             password_hash,

From 019ccda845e69262c2e39fa79b2cd0da85f0ca0c Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 14:50:05 -0400
Subject: [PATCH 09/10] Rename variable

---
 server/src/api/tracks.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/server/src/api/tracks.rs b/server/src/api/tracks.rs
index b168c2d..5d01ea5 100644
--- a/server/src/api/tracks.rs
+++ b/server/src/api/tracks.rs
@@ -32,7 +32,7 @@ async fn get_track_check_user(
     track_id: i32,
     user: &users::Model,
 ) -> Result<Json<tracks::Model>, Either<Status, api::ErrorResponder>> {
-    if let Some(Some(user)) = user
+    if let Some(Some(track)) = user
         .find_related(Tracks)
         .filter(tracks::Column::Id.eq(track_id))
         .one(db)
@@ -40,7 +40,7 @@ async fn get_track_check_user(
         .transpose()
         .map(|it| it.ok())
     {
-        Ok(Json(user))
+        Ok(Json(track))
     } else {
         Err(Left(Status::NotFound))
     }

From 033f2a561baa3272a4dbadbcbb2e0cdd7eb713ca Mon Sep 17 00:00:00 2001
From: "D. Scott Boggs" <scott@tams.tech>
Date: Sat, 26 Aug 2023 14:51:14 -0400
Subject: [PATCH 10/10] Nit

---
 server/src/api/update.rs | 1 +
 1 file changed, 1 insertion(+)

diff --git a/server/src/api/update.rs b/server/src/api/update.rs
index b68204d..7cacb7b 100644
--- a/server/src/api/update.rs
+++ b/server/src/api/update.rs
@@ -95,6 +95,7 @@ impl Update {
         if receiver_count > 0 {
             trace!(receiver_count = receiver_count, update = as_serde!(self); "sending update");
             let count = tx.send(self.clone())?;
+            trace!(count = count; "update sent");
         } else {
             trace!("no update receivers, skipping message");
         }