Add runtime check in ArtifactPath::new

This adds a runtime check whether the artifact path is indeed relative.
An absolute artifact path is a bug.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>

1 files changed, 8 insertions, 6 deletions

diff --git a/src/filestore/staging.rs b/src/filestore/staging.rs
index df7e006..93c4047 100644
--- a/src/filestore/staging.rs
+++ b/src/filestore/staging.rs
@@ -70,12 +70,14 @@ impl StagingStore {
                     None
                 } else {
                     Some({
-                        let path = ArtifactPath::new(path);
-                        self.0.load_from_path(&path)
-                            .inspect(|r| trace!("Loaded from path {} = {:?}", path.display(), r))
-                            .with_context(|| anyhow!("Loading from path: {}", path.display()))
-                            .map_err(Error::from)
-                            .map(|art| art.path().clone())
+                        ArtifactPath::new(path)
+                            .and_then(|ap| {
+                                self.0.load_from_path(&ap)
+                                    .inspect(|r| trace!("Loaded from path {} = {:?}", ap.display(), r))
+                                    .with_context(|| anyhow!("Loading from path: {}", ap.display()))
+                                    .map_err(Error::from)
+                                    .map(|art| art.path().clone())
+                            })
                     })
                 }
             })