Compare commits

..

No commits in common. "c94915e35120850a0de59ab125a929fd0c066945" and "7758385b512b85d0042ffef9720a8d12eaeeeb55" have entirely different histories.

18 changed files with 57 additions and 187 deletions

2
.gitignore vendored
View file

@ -1,3 +1 @@
/target
*.snap.new
*.pending-snap

View file

@ -63,8 +63,6 @@ pub enum ApiError {
InvalidArchiveType,
#[error("invalid color")]
InvalidColor,
#[error("join error: {0}")]
TokioJoin(#[from] tokio::task::JoinError),
}
impl ResponseError for ApiError {
@ -75,7 +73,6 @@ impl ResponseError for ApiError {
| ApiError::InvalidArchiveType
| ApiError::InvalidColor => StatusCode::BAD_REQUEST,
ApiError::NoAccess => StatusCode::FORBIDDEN,
ApiError::TokioJoin(_) => StatusCode::INTERNAL_SERVER_ERROR,
}
}
}
@ -102,17 +99,11 @@ impl TalonApi {
&self,
talon: Data<&Talon>,
subdomain: Path<String>,
) -> Result<Response<Json<Website>>> {
) -> Result<Json<Website>> {
talon
.db
.get_website(&subdomain)
.map(|website| {
let modified = website.updated_at();
Response::new(Json(Website::from((subdomain.0, website)))).header(
header::LAST_MODIFIED,
httpdate::fmt_http_date(modified.into()),
)
})
.map(|w| Json(Website::from((subdomain.0, w))))
.map_err(Error::from)
}
@ -172,14 +163,9 @@ impl TalonApi {
) -> Result<()> {
auth.check_subdomain(&subdomain, Access::Modify)?;
let t2 = talon.clone();
let sd = subdomain.clone();
tokio::task::spawn_blocking(move || {
t2.icons.insert_icon(Cursor::new(data.as_slice()), &sd)
})
.await
.map_err(ApiError::from)??;
talon
.icons
.insert_icon(Cursor::new(data.as_slice()), &subdomain)?;
talon.db.update_website(
&subdomain,
db::model::WebsiteUpdate {
@ -266,8 +252,7 @@ impl TalonApi {
/// Mimimum visibility of the websites
#[oai(default)]
visibility: Query<Visibility>,
) -> Result<Response<Json<Vec<Website>>>> {
let modified = talon.db.websites_last_update()?;
) -> Result<Json<Vec<Website>>> {
talon
.db
.get_websites()
@ -285,10 +270,7 @@ impl TalonApi {
Err(_) => true,
})
.collect::<Result<Vec<_>, _>>()
.map(|data| {
Response::new(Json(data))
.header(header::LAST_MODIFIED, httpdate::fmt_http_date(modified))
})
.map(Json)
.map_err(Error::from)
}
@ -298,19 +280,14 @@ impl TalonApi {
&self,
talon: Data<&Talon>,
subdomain: Path<String>,
) -> Result<Response<Json<Vec<Version>>>> {
let website = talon.db.get_website(&subdomain)?;
) -> Result<Json<Vec<Version>>> {
talon.db.website_exists(&subdomain)?;
talon
.db
.get_website_versions(&subdomain)
.map(|r| r.map(Version::from))
.collect::<Result<Vec<_>, _>>()
.map(|data| {
Response::new(Json(data)).header(
header::LAST_MODIFIED,
httpdate::fmt_http_date(website.updated_at().into()),
)
})
.map(Json)
.map_err(Error::from)
}
@ -321,17 +298,11 @@ impl TalonApi {
talon: Data<&Talon>,
subdomain: Path<String>,
version: Path<u32>,
) -> Result<Response<Json<Version>>> {
) -> Result<Json<Version>> {
talon
.db
.get_version(&subdomain, *version)
.map(|v| {
let create_date = v.created_at;
Response::new(Json(Version::from((*version, v)))).header(
header::LAST_MODIFIED,
httpdate::fmt_http_date(create_date.into()),
)
})
.map(|v| Json(Version::from((*version, v))))
.map_err(Error::from)
}
@ -342,19 +313,14 @@ impl TalonApi {
talon: Data<&Talon>,
subdomain: Path<String>,
version: Path<u32>,
) -> Result<Response<Json<Vec<VersionFile>>>> {
let v = talon.db.get_version(&subdomain, *version)?;
) -> Result<Json<Vec<VersionFile>>> {
talon.db.version_exists(&subdomain, *version)?;
talon
.db
.get_version_files(&subdomain, *version)
.map(|r| r.map(VersionFile::from))
.collect::<Result<Vec<_>, _>>()
.map(|r| {
Response::new(Json(r)).header(
header::LAST_MODIFIED,
httpdate::fmt_http_date(v.created_at.into()),
)
})
.map(Json)
.map_err(Error::from)
}
@ -414,21 +380,15 @@ impl TalonApi {
// Try to store the uploaded website
// If this fails, the new version needs to be deleted
fn try_insert(
talon: &Talon,
data: Binary<Vec<u8>>,
subdomain: &str,
version: u32,
fallback: Option<String>,
) -> Result<()> {
let try_insert = || {
if data.starts_with(&hex!("1f8b")) {
talon
.storage
.insert_tgz_archive(data.as_slice(), subdomain, version)?;
.insert_tgz_archive(data.as_slice(), &subdomain, version)?;
} else if data.starts_with(&hex!("504b0304")) {
talon.storage.insert_zip_archive(
Cursor::new(data.as_slice()),
subdomain,
&subdomain,
version,
)?;
} else {
@ -436,26 +396,20 @@ impl TalonApi {
}
// Validata fallback path
if let Some(fallback) = &fallback {
if let Some(fallback) = &fallback.0 {
if let Err(e) =
talon
.storage
.get_file(subdomain, version, fallback, &Default::default())
.get_file(&subdomain, version, fallback, &Default::default())
{
return Err(Error::from(ApiError::InvalidFallback(e.to_string())));
}
}
Ok(())
}
};
let t2 = talon.clone();
let sd = subdomain.clone();
match tokio::task::spawn_blocking(move || try_insert(&t2, data, &sd, version, fallback.0))
.await
.map_err(|e| Error::from(ApiError::from(e)))
{
Ok(Ok(())) => {
match try_insert() {
Ok(()) => {
talon.db.update_website(
&subdomain,
db::model::WebsiteUpdate {
@ -465,7 +419,7 @@ impl TalonApi {
)?;
Ok(())
}
Err(e) | Ok(Err(e)) => {
Err(e) => {
// Remove the bad version and decrement the id counter
let _ = talon.db.delete_version(&subdomain, version, false);
let _ = talon.db.decrement_vid(&subdomain, version);

View file

@ -208,7 +208,6 @@ impl Db {
w.source_url = website.source_url.unwrap_or(w.source_url);
w.source_icon = website.source_icon.unwrap_or(w.source_icon);
w.has_icon = website.has_icon.unwrap_or(w.has_icon);
w.updated_at = Some(OffsetDateTime::now_utc());
rmp_serde::to_vec(&w).ok()
}

View file

@ -29,19 +29,13 @@ pub struct Website {
/// Does the website have an icon?
#[serde(default)]
pub has_icon: bool,
/// Website update date
#[serde(default)]
pub updated_at: Option<OffsetDateTime>,
}
impl Default for Website {
fn default() -> Self {
let created_at = OffsetDateTime::now_utc();
Self {
name: Default::default(),
created_at,
updated_at: Some(created_at),
created_at: OffsetDateTime::now_utc(),
latest_version: Default::default(),
color: Default::default(),
visibility: Default::default(),
@ -53,12 +47,6 @@ impl Default for Website {
}
}
impl Website {
pub fn updated_at(&self) -> OffsetDateTime {
self.updated_at.unwrap_or(self.created_at)
}
}
/// Update a website in the database with the contained values
///
/// Values set to `None` remain unchanged.

View file

@ -37,7 +37,7 @@ impl ResponseError for ImagesError {
}
}
const IMAGE_SIZE: u32 = 48;
const IMAGE_SIZE: u32 = 32;
const MAX_IMAGE_SIZE: u32 = 4000;
type Result<T> = std::result::Result<T, ImagesError>;

View file

@ -75,7 +75,6 @@ fn insert_websites(db: &Db) {
&Website {
name: "ThetaDev".to_owned(),
created_at: datetime!(2023-02-18 16:30 +0),
updated_at: Some(datetime!(2023-02-18 16:30 +0)),
latest_version: Some(2),
color: Some(2068974),
visibility: talon::model::Visibility::Featured,
@ -88,7 +87,6 @@ fn insert_websites(db: &Db) {
&Website {
name: "Spotify-Gender-Ex".to_owned(),
created_at: datetime!(2023-02-18 16:30 +0),
updated_at: Some(datetime!(2023-02-18 16:30 +0)),
latest_version: Some(1),
color: Some(1947988),
visibility: talon::model::Visibility::Featured,
@ -103,7 +101,6 @@ fn insert_websites(db: &Db) {
&Website {
name: "RustyPipe".to_owned(),
created_at: datetime!(2023-02-20 18:30 +0),
updated_at: Some(datetime!(2023-02-20 18:30 +0)),
latest_version: Some(1),
color: Some(7943647),
visibility: talon::model::Visibility::Featured,
@ -118,7 +115,6 @@ fn insert_websites(db: &Db) {
&Website {
name: "SvelteKit SPA".to_owned(),
created_at: datetime!(2023-03-03 22:00 +0),
updated_at: Some(datetime!(2023-03-03 22:00 +0)),
latest_version: Some(1),
color: Some(16727552),
visibility: talon::model::Visibility::Hidden,

View file

@ -2,9 +2,9 @@
source: tests/tests.rs
expression: data
---
{"type":"website","key":"rustypipe","value":{"name":"RustyPipe","created_at":[2023,51,18,30,0,0,0,0,0],"latest_version":1,"color":7943647,"visibility":"featured","source_url":"https://code.thetadev.de/ThetaDev/rustypipe","source_icon":"gitea","vid_count":1,"has_icon":false,"updated_at":[2023,51,18,30,0,0,0,0,0]}}
{"type":"website","key":"spa","value":{"name":"SvelteKit SPA","created_at":[2023,62,22,0,0,0,0,0,0],"latest_version":1,"color":16727552,"visibility":"hidden","source_url":null,"source_icon":null,"vid_count":1,"has_icon":false,"updated_at":[2023,62,22,0,0,0,0,0,0]}}
{"type":"website","key":"spotify-gender-ex","value":{"name":"Spotify-Gender-Ex","created_at":[2023,49,16,30,0,0,0,0,0],"latest_version":1,"color":1947988,"visibility":"featured","source_url":"https://github.com/Theta-Dev/Spotify-Gender-Ex","source_icon":"github","vid_count":1,"has_icon":false,"updated_at":[2023,49,16,30,0,0,0,0,0]}}
{"type":"website","key":"rustypipe","value":{"name":"RustyPipe","created_at":[2023,51,18,30,0,0,0,0,0],"latest_version":1,"color":7943647,"visibility":"featured","source_url":"https://code.thetadev.de/ThetaDev/rustypipe","source_icon":"gitea","vid_count":1,"has_icon":false}}
{"type":"website","key":"spa","value":{"name":"SvelteKit SPA","created_at":[2023,62,22,0,0,0,0,0,0],"latest_version":1,"color":16727552,"visibility":"hidden","source_url":null,"source_icon":null,"vid_count":1,"has_icon":false}}
{"type":"website","key":"spotify-gender-ex","value":{"name":"Spotify-Gender-Ex","created_at":[2023,49,16,30,0,0,0,0,0],"latest_version":1,"color":1947988,"visibility":"featured","source_url":"https://github.com/Theta-Dev/Spotify-Gender-Ex","source_icon":"github","vid_count":1,"has_icon":false}}
{"type":"version","key":"rustypipe:1","value":{"created_at":[2023,51,18,30,0,0,0,0,0],"data":{},"fallback":null,"spa":false}}
{"type":"version","key":"spa:1","value":{"created_at":[2023,62,22,0,0,0,0,0,0],"data":{},"fallback":"200.html","spa":true}}
{"type":"version","key":"spotify-gender-ex:1","value":{"created_at":[2023,49,16,30,0,0,0,0,0],"data":{},"fallback":null,"spa":false}}

View file

@ -2,10 +2,10 @@
source: tests/tests.rs
expression: data
---
{"type":"website","key":"-","value":{"name":"ThetaDev","created_at":[2023,49,16,30,0,0,0,0,0],"latest_version":2,"color":2068974,"visibility":"featured","source_url":null,"source_icon":null,"vid_count":2,"has_icon":false,"updated_at":[2023,49,16,30,0,0,0,0,0]}}
{"type":"website","key":"rustypipe","value":{"name":"RustyPipe","created_at":[2023,51,18,30,0,0,0,0,0],"latest_version":1,"color":7943647,"visibility":"featured","source_url":"https://code.thetadev.de/ThetaDev/rustypipe","source_icon":"gitea","vid_count":1,"has_icon":false,"updated_at":[2023,51,18,30,0,0,0,0,0]}}
{"type":"website","key":"spa","value":{"name":"SvelteKit SPA","created_at":[2023,62,22,0,0,0,0,0,0],"latest_version":1,"color":16727552,"visibility":"hidden","source_url":null,"source_icon":null,"vid_count":1,"has_icon":false,"updated_at":[2023,62,22,0,0,0,0,0,0]}}
{"type":"website","key":"spotify-gender-ex","value":{"name":"Spotify-Gender-Ex","created_at":[2023,49,16,30,0,0,0,0,0],"latest_version":1,"color":1947988,"visibility":"featured","source_url":"https://github.com/Theta-Dev/Spotify-Gender-Ex","source_icon":"github","vid_count":1,"has_icon":false,"updated_at":[2023,49,16,30,0,0,0,0,0]}}
{"type":"website","key":"-","value":{"name":"ThetaDev","created_at":[2023,49,16,30,0,0,0,0,0],"latest_version":2,"color":2068974,"visibility":"featured","source_url":null,"source_icon":null,"vid_count":2,"has_icon":false}}
{"type":"website","key":"rustypipe","value":{"name":"RustyPipe","created_at":[2023,51,18,30,0,0,0,0,0],"latest_version":1,"color":7943647,"visibility":"featured","source_url":"https://code.thetadev.de/ThetaDev/rustypipe","source_icon":"gitea","vid_count":1,"has_icon":false}}
{"type":"website","key":"spa","value":{"name":"SvelteKit SPA","created_at":[2023,62,22,0,0,0,0,0,0],"latest_version":1,"color":16727552,"visibility":"hidden","source_url":null,"source_icon":null,"vid_count":1,"has_icon":false}}
{"type":"website","key":"spotify-gender-ex","value":{"name":"Spotify-Gender-Ex","created_at":[2023,49,16,30,0,0,0,0,0],"latest_version":1,"color":1947988,"visibility":"featured","source_url":"https://github.com/Theta-Dev/Spotify-Gender-Ex","source_icon":"github","vid_count":1,"has_icon":false}}
{"type":"version","key":"-:1","value":{"created_at":[2023,49,16,30,0,0,0,0,0],"data":{"Deployed by":"https://github.com/Theta-Dev/Talon/actions/runs/1352014628","Version":"v0.1.0"},"fallback":null,"spa":false}}
{"type":"version","key":"-:2","value":{"created_at":[2023,49,16,52,0,0,0,0,0],"data":{"Deployed by":"https://github.com/Theta-Dev/Talon/actions/runs/1354755231","Version":"v0.1.1"},"fallback":null,"spa":false}}
{"type":"version","key":"rustypipe:1","value":{"created_at":[2023,51,18,30,0,0,0,0,0],"data":{},"fallback":null,"spa":false}}

View file

@ -13,7 +13,6 @@ expression: "vec![ws1, ws2, ws3]"
source_icon: None,
vid_count: 2,
has_icon: false,
updated_at: Some((2023, 49, 16, 30, 0, 0, 0, 0, 0)),
),
Website(
name: "Spotify-Gender-Ex",
@ -25,7 +24,6 @@ expression: "vec![ws1, ws2, ws3]"
source_icon: Some(github),
vid_count: 1,
has_icon: false,
updated_at: Some((2023, 49, 16, 30, 0, 0, 0, 0, 0)),
),
Website(
name: "RustyPipe",
@ -37,6 +35,5 @@ expression: "vec![ws1, ws2, ws3]"
source_icon: Some(gitea),
vid_count: 1,
has_icon: false,
updated_at: Some((2023, 51, 18, 30, 0, 0, 0, 0, 0)),
),
]

View file

@ -13,7 +13,6 @@ expression: websites
source_icon: None,
vid_count: 2,
has_icon: false,
updated_at: Some((2023, 49, 16, 30, 0, 0, 0, 0, 0)),
)),
("rustypipe", Website(
name: "RustyPipe",
@ -25,7 +24,6 @@ expression: websites
source_icon: Some(gitea),
vid_count: 1,
has_icon: false,
updated_at: Some((2023, 51, 18, 30, 0, 0, 0, 0, 0)),
)),
("spa", Website(
name: "SvelteKit SPA",
@ -37,7 +35,6 @@ expression: websites
source_icon: None,
vid_count: 1,
has_icon: false,
updated_at: Some((2023, 62, 22, 0, 0, 0, 0, 0, 0)),
)),
("spotify-gender-ex", Website(
name: "Spotify-Gender-Ex",
@ -49,6 +46,5 @@ expression: websites
source_icon: Some(github),
vid_count: 1,
has_icon: false,
updated_at: Some((2023, 49, 16, 30, 0, 0, 0, 0, 0)),
)),
]

View file

@ -12,5 +12,4 @@ Website(
source_icon: Some(link),
vid_count: 2,
has_icon: false,
updated_at: "[date]",
)

View file

@ -8,8 +8,6 @@ use rstest::rstest;
use fixtures::*;
use talon::db::{Db, DbError};
const ICON_SIZE: u32 = 48;
mod database {
use super::*;
@ -85,7 +83,7 @@ mod database {
.unwrap();
let website = db.get_website(SUBDOMAIN_1).unwrap();
insta::assert_ron_snapshot!(website, {".updated_at" => "[date]"});
insta::assert_ron_snapshot!(website);
}
#[rstest]
@ -615,8 +613,8 @@ mod icons {
assert!(stored_path.is_file());
let stored_img = ImageReader::open(&stored_path).unwrap().decode().unwrap();
assert_eq!(stored_img.height(), ICON_SIZE);
assert_eq!(stored_img.width(), ICON_SIZE);
assert_eq!(stored_img.height(), 32);
assert_eq!(stored_img.width(), 32);
}
#[test]
@ -858,7 +856,7 @@ mod api {
resp.assert_status_is_ok();
let ws = tln.db.get_website("test").unwrap();
insta::assert_ron_snapshot!(ws, {".created_at" => "[date]", ".updated_at" => "[date]"}, @r###"
insta::assert_ron_snapshot!(ws, {".created_at" => "[date]"}, @r###"
Website(
name: "Test",
created_at: "[date]",
@ -869,7 +867,6 @@ mod api {
source_icon: Some(git),
vid_count: 0,
has_icon: false,
updated_at: "[date]",
)
"###);
}
@ -914,7 +911,7 @@ mod api {
resp.assert_status_is_ok();
let ws = tln.db.get_website("-").unwrap();
insta::assert_ron_snapshot!(ws, {".updated_at" => "[date]"}, @r###"
insta::assert_ron_snapshot!(ws, @r###"
Website(
name: "Test",
created_at: (2023, 49, 16, 30, 0, 0, 0, 0, 0),
@ -925,7 +922,6 @@ mod api {
source_icon: Some(git),
vid_count: 2,
has_icon: false,
updated_at: "[date]",
)
"###);
}
@ -971,8 +967,8 @@ mod api {
.unwrap()
.decode()
.unwrap();
assert_eq!(got_icon.height(), ICON_SIZE);
assert_eq!(got_icon.width(), ICON_SIZE);
assert_eq!(got_icon.height(), 32);
assert_eq!(got_icon.width(), 32);
}
#[rstest]

View file

@ -9,6 +9,7 @@
let currentWebsite: Website;
currentWebsiteStore.subscribe((ws) => {
console.log("current ws changed", ws);
currentWebsite = ws;
});

View file

@ -139,7 +139,7 @@
height: 100%
z-index: 999999
padding: 3em 0.4em 0.4em
padding: 1em 0.4em
display: flex
flex-direction: column

View file

@ -2,7 +2,6 @@
import PageIcon from "./PageIcon.svelte";
import {
formatDate,
getSubdomainAndVersion,
getWebsiteVersionUrl,
isUrl,
trimCommit,
@ -15,37 +14,28 @@
import Modal from "./Modal.svelte";
import { openModal } from "svelte-modals";
import InstanceInfoModal from "./InstanceInfoModal.svelte";
import { onMount } from "svelte";
let currentWebsite: Website;
currentWebsiteStore.subscribe((ws) => {
currentWebsite = ws;
});
const currentVid: number | null = getSubdomainAndVersion()[1];
export let isOpen: boolean;
onMount(async () => {
const v = await client.websiteSubdomainVersionsGet({
subdomain: currentWebsite.subdomain,
});
$: {
if (isOpen && currentWebsite) {
client
.websiteSubdomainVersionsGet({ subdomain: currentWebsite.subdomain })
.then((v) => {
versions = v;
if (v && v.length > 0) {
latestVersion = v[v.length - 1];
if (currentVid !== null) {
currentVersion = v.find((v) => v.id == currentVid);
} else {
currentVersion = latestVersion;
}
currentVersion = v[v.length - 1];
}
});
}
}
let versions: Version[] = [];
let currentVersion: Version | undefined;
let latestVersion: Version | undefined;
let currentVersion: Version = null;
function getVersionAttr(version: Version): string | null {
return (
@ -73,14 +63,6 @@
<p class="divider">
<InlineIcon iconName="question" />
Current version #{currentVersion.id}
{#if latestVersion && latestVersion !== currentVersion}
<a
class="latest-tag"
href={getWebsiteVersionUrl(currentWebsite.subdomain, latestVersion.id)}
>Latest: #{latestVersion.id}</a
>
{/if}
</p>
<Tag key="Upload date" value={formatDate(currentVersion.createdAt)} />
@ -113,9 +95,7 @@
<div>
Powered by
<button class="link" on:click={openInstanceInfo}
>Talon {talonConfig.version}</button
>
<button on:click={openInstanceInfo}>Talon {talonConfig.version}</button>
</div>
{/if}
</Modal>
@ -131,13 +111,4 @@
font-size: 2em
margin-left: 0.25em
.latest-tag
background-color: lime
color: values.$color-text-d1
margin: 0 1em
overflow: hidden
white-space: nowrap
padding: 0 0.4em
border-radius: 1em
</style>

View file

@ -6,16 +6,15 @@
font-family: sans-serif
color: values.$color-text
a, .link
a, button
display: inline
text-decoration: none
cursor: pointer
background: none
border: none
box-shadow: none
padding: 0
.link
a
color: var(--talon-color)
filter: brightness(150%)

View file

@ -13,4 +13,3 @@ $color-base-2: color.scale($color-base, $lightness: 20%)
$color-primary-light: color.scale($color-primary, $lightness: 15%)
$color-primary-dark: color.scale($color-primary, $lightness: -15%)
$color-text-1: color.scale($color-text, $lightness: -15%)
$color-text-d1: color.scale($color-base, $lightness: -20%)

View file

@ -21,29 +21,6 @@ export function getSubdomain(): string {
return "-";
}
export function getSubdomainAndVersion(): [string, number | null] {
const hn = window.location.hostname;
const rd_noport = talonConfig.root_domain.split(":", 1)[0];
if (hn.endsWith("." + rd_noport)) {
const subdomainSplit = hn
.substring(0, hn.length - rd_noport.length - 1)
.split("--", 2);
const subdomain = subdomainSplit[0];
let version =
subdomainSplit.length > 1 ? parseInt(subdomainSplit[1].replace(/^v/, "")) : null;
if (Number.isNaN(version)) version = null;
if (subdomain === "x") {
return ["-", version];
} else {
return [subdomain, version];
}
}
return ["-", null];
}
export function getWebsiteUrl(subdomain: string): string {
const proto = window.location.protocol;