literal design flaw

Author: 0x7d8

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "api"
-version = "2.3.0"
+version = "2.3.1"
 edition = "2024"
 
 [profile.dev]

Cargo.toml

@@ -161,9 +161,9 @@ impl Extension {
             FROM extensions
             JOIN authors ON extensions.author_id = authors.id
             WHERE
-                identifier = $1
-                AND NOT pending
-                AND NOT hidden
+                extensions.identifier = $1
+                AND NOT extensions.pending
+                AND NOT extensions.hidden
             "#,
             Self::columns_sql(None, None)
         ))
@@ -182,9 +182,9 @@ impl Extension {
             FROM extensions
             JOIN authors ON extensions.author_id = authors.id
             WHERE
-                id = $1
-                AND NOT pending
-                AND NOT hidden
+                extensions.id = $1
+                AND NOT extensions.pending
+                AND NOT extensions.hidden
             "#,
             Self::columns_sql(None, None)
         ))

src/models/extension.rs

@@ -7,6 +7,8 @@ pub mod extension;
 
 pub trait BaseModel: Serialize + DeserializeOwned {
     fn columns(prefix: Option<&str>, table: Option<&str>) -> BTreeMap<String, String>;
+
+    #[inline]
     fn columns_sql(prefix: Option<&str>, table: Option<&str>) -> String {
         Self::columns(prefix, table)
             .iter()

src/models/mod.rs

@@ -131,7 +131,10 @@ impl TelemetryLogger {
     }
 
     #[inline]
-    async fn lookup_ips(&self, ips: &[&str]) -> HashMap<String, [String; 2]> {
+    async fn lookup_ips(
+        &self,
+        ips: &[&str],
+    ) -> Result<HashMap<String, [String; 2]>, reqwest::Error> {
         let mut result = HashMap::new();
 
         let data = self
@@ -149,11 +152,9 @@ impl TelemetryLogger {
                     .collect::<HashSet<_>>(),
             )
             .send()
-            .await
-            .unwrap()
+            .await?
             .json::<Vec<serde_json::Value>>()
-            .await
-            .unwrap();
+            .await?;
 
         for entry in data {
             if entry.get("continentCode").is_none() || entry.get("countryCode").is_none() {
@@ -169,7 +170,7 @@ impl TelemetryLogger {
             );
         }
 
-        result
+        Ok(result)
     }
 
     pub async fn process(&self) {
@@ -192,7 +193,8 @@ impl TelemetryLogger {
                     .collect::<Vec<_>>()
                     .as_slice(),
             )
-            .await;
+            .await
+            .unwrap_or_default();
 
         for t in telemetry.iter_mut() {
             if let Some([continent, country]) = ips.get(&t.ip) {