Skip to content
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.

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

Remove access token {key} handling, and let it be in the style url #1023

Closed
birkskyum opened this issue Jan 22, 2025 · 1 comment
Closed

Comments

@birkskyum
Copy link
Member

birkskyum commented Jan 22, 2025

There is a lot of complexity now associated to handling the access tokens, and it's causing bugs.

This ticket suggest to remove all that and just have the keys in the styles

Problems associated to the current handling

It seems there were some concerns of potential ability to misused the built-in access tokens here, with an export function:

@birkskyum
Copy link
Member Author

birkskyum commented Jan 22, 2025

Nevermind, it's convenient that the maputnik can load styles that has the {key} fields, like the ones stored on github.

Such as https://github.com/maputnik/osm-liberty

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

No branches or pull requests

1 participant