Skip to content

Remove Streamlit Mapbox API key #1136

@tvst

Description

@tvst

Edit: apparently this is not how Mapbox works. See Update section below.

If I understand correctly how Mapbox works, I think we don't need an API key by default since Mapbox throttles usage of their maps based on IP address (which is user-specific).

Can we just remove the default API key? And we'd need to allow users to enter their own API key, of course.


Update:

DeckGL depends on Mapbox to show maps. Mapbox requires developers to have a token. Up to now, Streamlit hard-coded a token to make people's lives easier. The problem is that for the past few months we've started going over the free QPS limit of that token, and so we've been paying for everyone's usage of maps.

So let's fix this!

Proposal:

  • Whenever a users tries to use DeckGL and they didn't provide a token (this is done via the config file), we'll show an error explaining what they need to do.
  • We'll update our docs to make this abundantly clear.
  • To make the streamlit hello experience nice, the map in that app will continue to use our own paid token. But we'll take steps to make sure only that app can use that token, as much as possible.

Metadata

Metadata

Assignees

Labels

type:enhancementRequests for feature enhancements or new features

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions