Skip to content

Fix for proper handling of relative links for assets of network-loaded STAC items #1599

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
gadomski merged 11 commits into from
Dec 12, 2025
Merged

Fix for proper handling of relative links for assets of network-loaded STAC items #1599

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e2629ea
fix for proper handling of absolute http URIs
spinto Dec 9, 2025
6061c6a
fix tests for windows
spinto Dec 9, 2025
962407a
added fix for handling of relative links to changelog
spinto Dec 10, 2025
2da6fae
Better function description
spinto Dec 11, 2025
9141676
Reworded changelog
spinto Dec 11, 2025
10f24e1
revert back change in item
spinto Dec 12, 2025
8d8a437
added explicit tests for get_absolute_href
spinto Dec 12, 2025
5e08cc8
use temporary path for get_absolute_href tests as windows does not li…
spinto Dec 12, 2025
aa039c1
More readable condition
spinto Dec 12, 2025
cf7f94b
separated windows and unix tests
spinto Dec 12, 2025
81891c5
fix lint checks
spinto Dec 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
revert back change in item
  • Loading branch information
@spinto
spinto committed Dec 12, 2025
commit 10f24e1b2114009795da04b88b4aa27f1f422efa
2 changes: 1 addition & 1 deletion pystac/item.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ def set_self_href(self, href: str | None) -> None:
# Make sure relative asset links remain valid.
for asset in self.assets.values():
asset_href = asset.href
if not is_absolute_href(asset_href, prev_href):
if not is_absolute_href(asset_href):
abs_href = make_absolute_href(asset_href, prev_href)
new_relative_href = make_relative_href(abs_href, new_href)
asset.href = new_relative_href
Expand Down