Fix returned archive name, actually return assets in build results

Signed-off-by: Robert Detjens <github@detjens.dev>

Author: detjensrobert

src/builder/artifacts.rs

@@ -176,7 +176,6 @@ async fn extract_rename(
 async fn extract_archive(
     chal: &ChallengeConfig,
     container: &docker::ContainerInfo,
-    // files: &Vec<PathBuf>,
     files: &[PathBuf],
     archive_name: &Path,
 ) -> Result<Vec<PathBuf>> {
@@ -203,7 +202,7 @@ async fn extract_archive(
     // archive_name already has the chal dir prepended
     zip_files(archive_name, &copied_files)?;
 
-    Ok(vec![chal.directory.join(archive_name)])
+    Ok(vec![archive_name.to_path_buf()])
 }
 
 /// Add multiple local `files` to a zipfile at `zip_name`

src/builder/mod.rs

@@ -138,7 +138,7 @@ async fn build_challenge(
 
         // extract each challenge provide entry
         // this handles both local files and from build containers
-        let extracted_files = chal
+        built.assets = chal
             .provide
             .iter()
             .map(|p| async {
@@ -152,7 +152,11 @@ async fn build_challenge(
                     })
             })
             .try_join_all()
-            .await?;
+            .await?
+            // flatten to single vec of all paths
+            .into_iter()
+            .flatten()
+            .collect_vec();
 
         info!("extracted artifacts: {:?}", built.assets);
     }