use printf instead of echo

Cargo.lock

@@ -410,7 +410,7 @@ dependencies = [
 
 [[package]]
 name = "artix-gitea"
-version = "0.1.3"
+version = "0.1.2"
 dependencies = [
  "actix-web",
  "awc",
@@ -420,7 +420,7 @@ dependencies = [
 
 [[package]]
 name = "artix-pkglib"
-version = "0.1.3"
+version = "0.1.2"
 dependencies = [
  "alpm",
  "alpm-utils",
@@ -430,7 +430,7 @@ dependencies = [
 
 [[package]]
 name = "artixweb_packages"
-version = "0.1.3"
+version = "0.1.2"
 dependencies = [
  "actix-files",
  "actix-identity",

Dockerfile.ci

@@ -46,11 +46,11 @@ RUN pacman -Sy --noconfirm postgresql-libs \
     && set -x \
     && pacman -Syu --noconfirm \
     && pacman -Scc --noconfirm \
-    && printf "[lib32]\nInclude = /etc/pacman.d/mirrorlist\n" >> /etc/pacman.conf \
-    && printf "[system-gremlins]\nInclude = /etc/pacman.d/mirrorlist\n" >> /etc/pacman.conf \
-    && printf "[world-gremlins]\nInclude = /etc/pacman.d/mirrorlist\n" >> /etc/pacman.conf \
-    && printf "[galaxy-gremlins]\nInclude = /etc/pacman.d/mirrorlist\n" >> /etc/pacman.conf \
-    && printf "[lib32-gremlins]\nInclude = /etc/pacman.d/mirrorlist\n" >> /etc/pacman.conf \
+    && printf "\n[lib32]\nInclude = /etc/pacman.d/mirrorlist" >> /etc/pacman.conf \
+    && printf "\n[system-gremlins]\nInclude = /etc/pacman.d/mirrorlist" >> /etc/pacman.conf \
+    && printf "\n[world-gremlins]\nInclude = /etc/pacman.d/mirrorlist" >> /etc/pacman.conf \
+    && printf "\n[galaxy-gremlins]\nInclude = /etc/pacman.d/mirrorlist" >> /etc/pacman.conf \
+    && printf "\n[lib32-gremlins]\nInclude = /etc/pacman.d/mirrorlist" >> /etc/pacman.conf \
     && pacman -Sy --noconfirm
 
 WORKDIR /usr/share/artixweb_packages

crates/artix-gitea/Cargo.toml

@@ -8,7 +8,7 @@ homepage = "https://packages.artixlinux.org"
 keywords = ["artix", "packages", "gitea"]
 license = "MIT OR Apache-2.0"
 edition = "2021"
-version = "0.1.3"
+version = "0.1.2"
 
 [lib]
 name = "artix_gitea"

crates/artix-pkglib/Cargo.toml

@@ -8,7 +8,7 @@ homepage = "https://packages.artixlinux.org"
 keywords = ["artix", "packages"]
 license = "MIT OR Apache-2.0"
 edition = "2021"
-version = "0.1.3"
+version = "0.1.2"
 
 [lib]
 name = "artix_pkglib"

crates/artixweb-packages/Cargo.toml

@@ -7,7 +7,7 @@ repository = "gitea.artixlinux.org/artix/artixweb_packages"
 keywords = ["artix", "packages"]
 license = "MIT OR Apache-2.0"
 edition = "2021"
-version = "0.1.3"
+version = "0.1.2"
 
 [[bin]]
 name = "artixweb_packages"
@@ -15,8 +15,8 @@ test = false
 bench = false
 
 [dependencies]
-artix-gitea = { path = "../artix-gitea", version = "=0.1.3" }
-artix-pkglib = { path = "../artix-pkglib", version = "=0.1.3" }
+artix-gitea = { path = "../artix-gitea", version = "=0.1.2" }
+artix-pkglib = { path = "../artix-pkglib", version = "=0.1.2" }
 
 actix-files = "0.6.0"
 actix-identity = "0.4"

crates/artixweb-packages/src/handlers/index.rs

@@ -121,31 +121,33 @@ pub async fn index(
         repos.join(":")
     };
 
-    let mut s = PackagesNavigation {
-        user_email,
-        packages: Vec::new(),
-        repos,
-        total: 0,
-        query: query_url.join("&"),
-        total_pages: 1,
-        generation_time: 0,
-        offset,
-        limit
-    };
-    if let Ok(result) =
+    let s = if let Ok(result) =
         get_packages_inner(&repos_criteria, limit, offset, search_criteria, Some(&pool)).await
     {
-        s.packages = result.0;
-        s.total = result.1;
-        s.total_pages = if limit > 0 {
-            (result.1 as f64 / limit as f64).ceil() as usize
-        } else {
-            1
-        };
-    }
-    s.generation_time = start_time.elapsed().as_millis();
+        PackagesNavigation {
+            user_email,
+            packages: result.0,
+            repos,
+            total: result.1,
+            query: query_url.join("&"),
+            total_pages: if limit > 0 {
+                (result.1 as f64 / limit as f64).ceil() as usize
+            } else {
+                1
+            },
+            generation_time: start_time.elapsed().as_millis(),
+            offset,
+            limit,
+        }
+        .render()
+        .unwrap()
+    } else {
+        return Ok(HttpResponse::NotFound()
+            .content_type("text/html")
+            .body("<html><body>404 - Page Not Found!</body></html>"));
+    };
 
-    Ok(HttpResponse::Ok().content_type("text/html").body(s.render().unwrap()))
+    Ok(HttpResponse::Ok().content_type("text/html").body(s))
 }
 
 #[allow(clippy::unnecessary_wraps)]

crates/artixweb-packages/src/handlers/packages.rs

@@ -457,7 +457,7 @@ pub(crate) async fn get_packages_details_inner(
                     .iter()
                     .map(std::string::ToString::to_string)
                     .collect(),
-                gitea_url: format!("{}/packages/{}", SETTINGS.gitea_url, pkg.name().to_string()),
+                gitea_url: String::new(),
                 dependencies: get_depends_from_package(
                     &handle,
                     pkg.arch().unwrap_or("any"),
@@ -517,6 +517,7 @@ pub(crate) async fn get_packages_details_inner(
                 web::block(move || get_package_metadata(&pkg_name_version, &pool)).await
             {
                 result.last_updated = metadata.last_update.timestamp();
+                result.gitea_url = metadata.gitea_url;
                 result.flagged = metadata.flagged;
                 if let Some(flagged_on) = metadata.flag_on {
                     result.flagged_on = flagged_on.timestamp_millis();

crates/artixweb-packages/templates/package_details.html

@@ -35,8 +35,8 @@
             <h2>{{ pkg.package_name }}-{{ pkg.version }} {% if pkg.flagged %}(<span class="flagged">flagged</span>){% endif %}</h2>
             <section role="action_panel" class="action_panel">
                 <h4>Actions Panel</h4>
-                <section role="action"><a href="{{ pkg.gitea_url }}">View Package Sources</a></section>
-                <section role="action"><a href="{{ pkg.gitea_url }}/graph">View Package Changes</a></section>
+                <section role="action"><a href="{{ pkg.gitea_url }}" target="blank">View Package Sources</a></section>
+                <section role="action"><a href="{{ pkg.gitea_url }}/graph" target="blank">View Package Changes</a></section>
                 {% if !pkg.flagged %}
                     <section role="action"><a href="/flag_package/{{ pkg.package_name }}/{{ pkg.version }}">Flag package out-of-date</a></section>
                 {% else %}
@@ -65,7 +65,7 @@
                     <section role="value">{{ pkg.replaces|provides_or_replaces }}</section>
                 {% endif %}
                 <section role="key">Upstream URL:</section>
-                <section role="value"><a href="{{ pkg.upstream_url }}">{{ pkg.upstream_url }}</a></section>
+                <section role="value"><a href="{{ pkg.upstream_url }}" target="_blank">{{ pkg.upstream_url }}</a></section>
                 <section role="key">Size:</section>
                 <section role="value">{{ pkg.size|human_readable }} MB</section>
                 <section role="key">Installed Size:</section>