web/src/app.rs

@@ -1,5 +1,6 @@
 use leptos::*;
 use leptos_router::*;
+use reqwest::Method;
 use crate::prelude::*;
 
 use leptos_use::{storage::use_local_storage, use_cookie, utils::{FromToStringCodec, JsonCodec}};
@@ -35,21 +36,36 @@ pub fn App() -> impl IntoView {
 
 	let title_target = move || if auth.present() { "/web/home" } else { "/web/server" };
 
-	local_tl.more(auth); // public outbox never contains private posts
+	if let Some(tok) = token.get_untracked() {
 		spawn_local(async move {
 			// refresh token first, or verify that we're still authed
-		if Auth::refresh(auth.token, set_token, set_userid).await {
-			home_tl.more(auth); // home inbox requires auth to be read
+			match reqwest::Client::new()
+				.request(Method::PATCH, format!("{URL_BASE}/auth"))
+				.json(&serde_json::json!({"token": tok}))
+				.send()
+				.await
+			{
+				Err(e) => tracing::error!("could not refresh token: {e}"),
+				Ok(res) => match res.error_for_status() {
+					Err(e) => tracing::error!("server rejected refresh: {e}"),
+					Ok(doc) => match doc.json::<AuthResponse>().await {
+						Err(e) => tracing::error!("failed parsing auth response: {e}"),
+						Ok(auth) => {
+							set_token.set(Some(auth.token));
+							set_userid.set(Some(auth.user));
+						},
+					}
+				}
 			}
-		server_tl.more(auth); // server inbox may contain private posts
-	});
 
-
-	// refresh token every hour
-	set_interval(
-		move || spawn_local(async move { Auth::refresh(auth.token, set_token, set_userid).await; }),
-		std::time::Duration::from_secs(3600)
-	);
+			server_tl.more(auth);
+			local_tl.more(auth);
+			if auth.token.get_untracked().is_some() { home_tl.more(auth) };
+		})
+	} else {
+		server_tl.more(auth);
+		local_tl.more(auth);
+	}
 
 	view! {
 		<nav class="w-100 mt-1 mb-1 pb-s">

web/src/auth.rs

@@ -1,6 +1,13 @@
 use leptos::*;
-use reqwest::Method;
-use crate::{components::AuthResponse, URL_BASE};
+use crate::URL_BASE;
+
+pub trait AuthToken {
+	fn present(&self) -> bool;
+	fn token(&self) -> String;
+	fn user_id(&self) -> String;
+	fn username(&self) -> String;
+	fn outbox(&self) -> String;
+}
 
 #[derive(Debug, Clone, Copy)]
 pub struct Auth {
@@ -8,16 +15,16 @@ pub struct Auth {
 	pub userid: Signal<Option<String>>,
 }
 
-impl Auth {
-	pub fn token(&self) -> String {
+impl AuthToken for Auth {
+	fn token(&self) -> String {
 		self.token.get().unwrap_or_default()
 	}
 
-	pub fn user_id(&self) -> String {
+	fn user_id(&self) -> String {
 		self.userid.get().unwrap_or_default()
 	}
 	
-	pub fn username(&self) -> String {
+	fn username(&self) -> String {
 		// TODO maybe cache this?? how often do i need it?
 		self.userid.get()
 			.unwrap_or_default()
@@ -27,40 +34,11 @@ impl Auth {
 			.to_string()
 	}
 
-	pub fn present(&self) -> bool {
+	fn present(&self) -> bool {
 		self.token.get().map_or(false, |x| !x.is_empty())
 	}
 
-	pub fn outbox(&self) -> String {
+	fn outbox(&self) -> String {
 		format!("{URL_BASE}/actors/{}/outbox", self.username())
 	}
-
-	pub async fn refresh(
-		token: Signal<Option<String>>,
-		set_token: WriteSignal<Option<String>>,
-		set_userid: WriteSignal<Option<String>>
-	) -> bool {
-		if let Some(tok) = token.get_untracked() {
-			match reqwest::Client::new()
-				.request(Method::PATCH, format!("{URL_BASE}/auth"))
-				.json(&serde_json::json!({"token": tok}))
-				.send()
-				.await
-			{
-				Err(e) => tracing::error!("could not refresh token: {e}"),
-				Ok(res) => match res.error_for_status() {
-					Err(e) => tracing::error!("server rejected refresh: {e}"),
-					Ok(doc) => match doc.json::<AuthResponse>().await {
-						Err(e) => tracing::error!("failed parsing auth response: {e}"),
-						Ok(auth) => {
-							set_token.set(Some(auth.token));
-							set_userid.set(Some(auth.user));
-							return true;
-						},
-					}
-				}
-			}
-		}
-		false
-	}
 }

web/src/components/timeline.rs

@@ -83,23 +83,21 @@ pub fn TimelineRepliesRecursive(tl: Timeline, root: String) -> impl IntoView {
 	let root_values = move || tl.feed
 		.get()
 		.into_iter()
+		.filter_map(|x| CACHE.get(&x))
 		.filter_map(|x| {
-			let document = CACHE.get(&x)?;
-			let (oid, reply) = match document.object_type().ok()? {
+			let oid = match x.object_type().ok()? {
 				// if it's a create, get and check created object: does it reply to root?
-				apb::ObjectType::Activity(apb::ActivityType::Create) => {
-					let object = CACHE.get(document.object().id().ok()?)?;
-					(object.id().str()?, object.in_reply_to().id().str()?)
-				},
+				apb::ObjectType::Activity(apb::ActivityType::Create) =>
+					CACHE.get(x.object().id().ok()?)?.in_reply_to().id().str()?,
 
 				// if it's a raw note, directly check if it replies to root
-				apb::ObjectType::Note => (document.id().str()?, document.in_reply_to().id().str()?),
+				apb::ObjectType::Note => x.in_reply_to().id().str()?,
 
 				// if it's anything else, check if it relates to root, maybe like or announce?
-				_ => (document.id().str()?, document.object().id().str()?),
+				_ => x.object().id().str()?,
 			};
-			if reply == root {
-				Some((oid, document))
+			if oid == root {
+				Some((oid, x))
 			} else {
 				None
 			}
@@ -110,7 +108,7 @@ pub fn TimelineRepliesRecursive(tl: Timeline, root: String) -> impl IntoView {
 		<For
 			each=root_values
 			key=|(id, _obj)| id.clone()
-			children=move |(id, obj)|
+			children=move |(id, obj)| {
 				view! {
 					<div class="context depth-r">
 						<Item item=obj replies=true />
@@ -119,6 +117,7 @@ pub fn TimelineRepliesRecursive(tl: Timeline, root: String) -> impl IntoView {
 						</div>
 					</div>
 				}
+			}
 		/ >
 	}
 }

web/src/page/object.rs

@@ -10,6 +10,8 @@ use apb::{Base, Object};
 pub fn ObjectPage(tl: Timeline) -> impl IntoView {
 	let params = use_params_map();
 	let auth = use_context::<Auth>().expect("missing auth context");
+	let id = params.get().get("id").cloned().unwrap_or_default();
+	let uid =  uriproxy::uri(URL_BASE, uriproxy::UriClass::Object, &id);
 	let object = create_local_resource(
 		move || params.get().get("id").cloned().unwrap_or_default(),
 		move |oid| async move {
@@ -36,7 +38,7 @@ pub fn ObjectPage(tl: Timeline) -> impl IntoView {
 			};
 			if let Ok(ctx) = obj.context().id() {
 				let tl_url = format!("{}/context/page", Uri::api(U::Object, ctx, false));
-				if !tl.next.get_untracked().starts_with(&tl_url) {
+				if !tl.next.get().starts_with(&tl_url) {
 					tl.reset(tl_url);
 				}
 			}
@@ -62,8 +64,7 @@ pub fn ObjectPage(tl: Timeline) -> impl IntoView {
 				{move || match object.get() {
 					None => view! { <p class="center"> loading ... </p> }.into_view(),
 					Some(None) => {
-						let raw_id = params.get().get("id").cloned().unwrap_or_default();
-						let uid =  uriproxy::uri(URL_BASE, uriproxy::UriClass::Object, &raw_id);
+						let uid = uid.clone();
 						view! { <p class="center"><code>loading failed</code><sup><small><a class="clean" href={uid} target="_blank">"↗"</a></small></sup></p> }.into_view()
 					},
 					Some(Some(o)) => {

web/src/page/user.rs

@@ -33,7 +33,7 @@ pub fn UserPage(tl: Timeline) -> impl IntoView {
 		move |id| {
 			async move {
 				let tl_url = format!("{}/outbox/page", Uri::api(U::Actor, &id, false));
-				if !tl.next.get_untracked().starts_with(&tl_url) {
+				if !tl.next.get().starts_with(&tl_url) {
 					tl.reset(tl_url);
 				}
 				match CACHE.get(&Uri::full(U::Actor, &id)) {