From abc793107c32c9da380638a2a7e0731848a39b96 Mon Sep 17 00:00:00 2001
From: sliu008 <69875423+sliu008@users.noreply.github.com>
Date: Tue, 5 Sep 2023 11:37:54 -0700
Subject: [PATCH] Feature/issue 172 (#193)

* fix shapefile subsetting

* update changelog
---
 CHANGELOG.md                       | 1 +
 podaac/subsetter/subset_harmony.py | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 2c7c881c..10cb9396 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 - [issue/182](https://github.com/podaac/l2ss-py/issues/182): Update code so doesn't remove '/' on attribute values.
 - [issue/178](https://github.com/podaac/l2ss-py/issues/178): Add function to make sure dimension in subset is same as original file
+- [issue/172](https://github.com/podaac/l2ss-py/issues/178): Fix shapefile subsetting by passing correct variable to subset function.
 ### Deprecated 
 ### Removed
 ### Fixed
diff --git a/podaac/subsetter/subset_harmony.py b/podaac/subsetter/subset_harmony.py
index d120f1cd..7dfc84ef 100644
--- a/podaac/subsetter/subset_harmony.py
+++ b/podaac/subsetter/subset_harmony.py
@@ -136,7 +136,7 @@ def process_item(self, item: pystac.Item, source: harmony.message.Source) -> pys
                 harmony_bbox = message.subset.bbox
 
             if message.subset and message.subset.shape:
-                subset_params['shapefile_path'] = download(
+                subset_params['shapefile'] = download(
                     message.subset.shape.href,
                     temp_dir,
                     logger=self.logger,