Skip to content
This repository has been archived by the owner on Jan 1, 2021. It is now read-only.

Use JOSM Imagery Sources #129

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

grischard
Copy link

No description provided.

@systemed
Copy link
Owner

systemed commented Apr 3, 2020

Thanks! I'll need to test this - Flash uses its own antique CORS-like method called crossdomain.xml, and although there is already a crossdomain.xml present on josm.openstreetmap.de, the whole thing is sufficiently arcane that I wouldn't be happy merging without testing. Once I've done that I'll merge this.

@systemed
Copy link
Owner

systemed commented May 8, 2020

I'm afraid this isn't mergeable as it stands, because https://josm.openstreetmap.de/crossdomain.xml only permits Flash to access resources on josm.openstreetmap.de if the SWF was loaded from an HTTPS page.

Although that wouldn't be a problem for openstreetmap.org, localhost (and aliases) are generally not HTTPS. In other words, this makes it impossible to develop P2 locally and show imagery, which means I don't know whether this patch would actually work or not. :)

If the JOSM folks add secure="false" attribute to the allow-access-from-domain element (cf http://osmlab.github.io/crossdomain.xml) then I believe this might work.

@grischard
Copy link
Author

grischard commented May 9, 2020

Gah.

Could testing on errol, maybe with a bit of sshfs, work?

@systemed
Copy link
Owner

systemed commented May 9, 2020

It could, but that would remove the ability to develop P2 locally, and require setting up a Flex toolchain on errol. JOSM adding 15 characters to one file on their server would be a lot easier.

@Firefishy
Copy link

Firefishy commented May 27, 2020

@systemed use localhost with a self-signed certificate.

Random selected guide: https://stackoverflow.com/questions/8169999/how-can-i-create-a-self-signed-cert-for-localhost

@systemed
Copy link
Owner

I think it's probably just easiest to mirror the JOSM file on systemed.net.

@grischard
Copy link
Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants