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

[Feature]: PMTiles Byte Range Request Support #7664

Closed
1 of 3 tasks
plasticviking opened this issue Sep 18, 2024 · 2 comments
Closed
1 of 3 tasks

[Feature]: PMTiles Byte Range Request Support #7664

plasticviking opened this issue Sep 18, 2024 · 2 comments

Comments

@plasticviking
Copy link

Description

Add .pmtiles to mediaExtensions in WebViewAssetHandler

Platforms

  • iOS
  • Android
  • Web

Request or proposed solution

Like audio and video files, PMTiles relies on HTTP 206 Partial Content HTTP responses (made with a range). See https://github.com/protomaps/PMTiles for the client-side library.

In a project I have been developing, I found that on the IOS platform I could not access bundled .pmtiles assets via range request (instead of partial content, full content was returned and triggered errors within the PMTiles client). I discovered after investigation that Capacitor IOS supports 206 responses for certain path extensions only and further that adding .pmtiles to the list of supported extensions resolves the problem and allows access to bundled large vector map tilesets in pmtiles format.

A PR accompanies this issue and passes this addition of the .pmtiles extension back upstream in case others have a requirement to use this mapping library.

Alternatives

No response

Additional Information

No response

@jcesarmobile
Copy link
Member

Can you provide a sample Capacitor app that uses PMTiles library?

@jcesarmobile jcesarmobile added the needs reply needs reply from the user label Sep 30, 2024
@ionitron-bot ionitron-bot bot removed the triage label Sep 30, 2024
@Ionitron
Copy link
Collaborator

It looks like this issue didn't get the information it needed, so I'll close it for now. If I made a mistake, sorry! I am just a bot.

Have a great day!
Ionitron 💙

@Ionitron Ionitron removed the needs reply needs reply from the user label Oct 16, 2024
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

3 participants