context_slug

Author: epompeii

lib/api_run/src/run.rs

@@ -1,5 +1,7 @@
 use bencher_endpoint::{CorsResponse, Endpoint, Post, ResponseCreated};
-use bencher_json::{project, system::auth, JsonNewRun, JsonReport, NameIdKind, ResourceName};
+use bencher_json::{
+    project, system::auth, JsonNewRun, JsonReport, NameIdKind, ReportContext, ResourceName,
+};
 use bencher_rbac::project::Permission;
 use bencher_schema::{
     conn_lock,
@@ -67,7 +69,10 @@ async fn post_inner(
             .map_err(|e| forbidden_error(e.to_string()))?;
             (auth_user, query_project)
         },
-        (Some(auth_user), Some(organization), None) => return Err(bad_request_error("todo")),
+        (Some(_auth_user), Some(_organization), None) => {
+            let _project_slug = json_run.context.as_ref().map(ReportContext::slug);
+            return Err(bad_request_error("Not yet supported"));
+        },
         _ => return Err(bad_request_error("Not yet supported")),
     };
 

lib/bencher_context/Cargo.toml

@@ -8,7 +8,7 @@ publish = false
 
 [features]
 client = ["dep:gix", "dep:uuid", "dep:windows"]
-server = ["bencher_valid/server"]
+server = ["dep:uuid", "bencher_valid/server"]
 schema = ["dep:schemars"]
 
 [dependencies]

lib/bencher_context/src/server/base36.rs

@@ -0,0 +1,37 @@
+use bencher_valid::BASE_36;
+use uuid::Uuid;
+
+pub fn encode_uuid(uuid: Uuid) -> String {
+    let base = BASE_36.len() as u128;
+    let chars = BASE_36.chars().collect::<Vec<_>>();
+
+    let mut num = uuid.as_u128();
+    let mut result = String::new();
+    while num > 0 {
+        let remainder = (num % base) as usize;
+        if let Some(c) = chars.get(remainder) {
+            result.push(*c);
+        }
+        num /= base;
+    }
+
+    result.chars().rev().collect()
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use uuid::Uuid;
+
+    #[test]
+    fn test_encode_uuid() {
+        let uuid = Uuid::parse_str("00000000-0000-0000-0000-000000000000").unwrap();
+        assert_eq!(encode_uuid(uuid), "");
+
+        let uuid = Uuid::parse_str("ffffffff-ffff-ffff-ffff-ffffffffffff").unwrap();
+        assert_eq!(encode_uuid(uuid), "f5lxx1zz5pnorynqglhzmsp33");
+
+        let uuid = Uuid::parse_str("12345678-1234-5678-1234-567812345678").unwrap();
+        assert_eq!(encode_uuid(uuid), "12srddy53kndus0lmbgjgy7i0");
+    }
+}

lib/bencher_context/src/server/base_36.rs

@@ -1,16 +1,32 @@
-use bencher_valid::Slug;
+use bencher_valid::{Slug, MAX_LEN};
+use uuid::Uuid;
 
 use crate::{ContextPath, ReportContext};
 
-mod base_36;
+mod base36;
 
 #[allow(clippy::multiple_inherent_impl)]
 impl ReportContext {
     pub fn slug(&self) -> Slug {
-        let name = self.repo_name();
-        let hash = self.repo_hash();
-
-        todo!()
+        // + 42 chars
+        let name = self.repo_name().map(truncate_name).unwrap_or_default();
+        // + 1 char (-)
+        // + 7 chars
+        let hash = self.repo_hash().map(short_hash).unwrap_or_default();
+        // + 1 char (-)
+        // + 13 chars
+        let fingerprint = self
+            .testbed_fingerprint()
+            .map(base36::encode_uuid)
+            .as_deref()
+            .map(short_fingerprint)
+            .unwrap_or_default();
+        // The spaces here are important,
+        // in case any of the values are empty
+        // they will essentially be ignored
+        let slug = format!("{name} {hash} {fingerprint}");
+        debug_assert!(slug.len() <= MAX_LEN, "Slug is too long: {slug}");
+        Slug::new(slug)
     }
 
     pub fn repo_name(&self) -> Option<&str> {
@@ -37,9 +53,19 @@ impl ReportContext {
         self.0.get(ContextPath::TESTBED_OS).map(String::as_str)
     }
 
-    pub fn testbed_fingerprint(&self) -> Option<&str> {
-        self.0
-            .get(ContextPath::TESTBED_FINGERPRINT)
-            .map(String::as_str)
+    pub fn testbed_fingerprint(&self) -> Option<Uuid> {
+        self.0.get(ContextPath::TESTBED_FINGERPRINT)?.parse().ok()
     }
 }
+
+fn truncate_name(name: &str) -> String {
+    name.chars().take(42).collect()
+}
+
+fn short_hash(hash: &str) -> String {
+    hash.chars().take(7).collect()
+}
+
+fn short_fingerprint(fingerprint: &str) -> String {
+    fingerprint.chars().take(13).collect()
+}

lib/bencher_context/src/server/mod.rs

@@ -60,7 +60,7 @@ pub use secret::Secret;
 pub use units::{Units, BYTES, NANOSECONDS, SECONDS};
 pub use user_name::UserName;
 
-const MAX_LEN: usize = 64;
+pub const MAX_LEN: usize = 64;
 
 #[cfg(feature = "wasm")]
 #[wasm_bindgen(start)]