Skip to content

See #24875: Point the overture source at the STAC catalog - #51

Merged
tsmock merged 1 commit into
JOSM:pagesfrom
watmildon:pages-overture-stac
Oct 7, 2026
Merged

tsmock merged 1 commit into
JOSM:pagesfrom
watmildon:pages-overture-stac

Conversation

@watmildon

Copy link
Copy Markdown
Contributor

The pmtiles catalog returns 403, which makes current plugin versions fail to load any default source. Plugin versions without STAC support read the new catalog as having no overture sources; versions with it list the latest release.

The pmtiles catalog returns 403, which makes current plugin versions fail to
load any default source. Plugin versions without STAC support read the new
catalog as having no overture sources; versions with it list the latest
release.
@stoecker

stoecker commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

I assume #52 obsoletes this?

@watmildon

Copy link
Copy Markdown
Contributor Author

We'll need this to feed into the machinery of #52. For folks on the old plugin, they're reader doesn't know what to do with the stac link but that results in skipping it, not erroring and causing an infinite "loading" message.

@watmildon

Copy link
Copy Markdown
Contributor Author

Do we want the STAC url to live in another place? Removing the entry or updating to this entry has materially the same effect on current consumers so I'm a bit confused.

tsmock added a commit that referenced this pull request Oct 7, 2026
…STAC"

This reverts commit a9ebb8e.
This is to make merging #51 easier.
@tsmock
tsmock merged commit 06af225 into JOSM:pages Oct 7, 2026
@tsmock

tsmock commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

We'll need this to feed into the machinery of #52. For folks on the old plugin, they're reader doesn't know what to do with the stac link but that results in skipping it, not erroring and causing an infinite "loading" message.

My typical response to old plugin problems that are now fixed, when the minimum JOSM and java versions do not change is that the user should update the plugin.

Since #52 does not increase the minimum Java or JOSM versions, not a problem.

I just removed the URL as a stop gap until I figured out what was going on (AKA stop the bleeding).

Its been a little while since I worked on JOSM/JOSM plugins.

@watmildon

Copy link
Copy Markdown
Contributor Author

All good. Just trying to make sure I've got a sense of things correctly in this foreign territory :-D

Thanks for working on things!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants