-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Remove Streamlit Mapbox API key #1136
Copy link
Copy link
Closed
Labels
type:enhancementRequests for feature enhancements or new featuresRequests for feature enhancements or new features
Description
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 helloexperience 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.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
type:enhancementRequests for feature enhancements or new featuresRequests for feature enhancements or new features