search

wavded / ogr2ogr

An ogr2ogr wrapper library
MIT License
214 stars 46 forks source link

/vsicurl/ can not be used with all kind of http or ftp resources #102

Closed jgrocha closed 11 months ago

jgrocha commented 1 year ago

/vsicurl/ prefix is being added to all resources matching /^(http|ftp)/

https://github.com/wavded/ogr2ogr/blob/729f321dd96414e256aa67be1bc76f0b0a9b21dc/index.ts#L105

There are several cases where /vsicurl/ can not be used and /vsicurl_streaming/ should be used instead. In my use case, I'm downloading WFS requests failing with /vsicurl/.

Users can add this prefix by themselves, if they want to. I don't think this should be added by the lib, behind the scenes.

Any comments against removing this from the library? I would like to discuss it before any PR.

wavded commented 1 year ago

Would it make sense to use /vsicurl_sreaming/ instead of /vsicurl/ ? Reason being is the API is more accessible if we don't require a user to know to append that to their FTP and HTTP endpoints. Or perhaps we allow an override for advanced uses?

github-actions[bot] commented 1 year ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] commented 11 months ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] commented 11 months ago

This issue was closed because it has been stalled for 5 days with no activity.