search

PDAL / PDAL

PDAL is Point Data Abstraction Library. GDAL for point cloud data.
https://pdal.io
Other
1.14k stars 446 forks source link

Https header support to readers #4523

Open trns1997 opened 1 month ago

trns1997 commented 1 month ago

For the moment based on usage, I know that reader.ept supports http headers that are passed to arbiter to read files in a remote server. I was wondering is there a particular reason why other readers do not have this option. I understand I could programmatically download for example my las file and pass the downloaded file to the LasReader, but I was thinking it would be cool to pass the readers the http headers with token and all necessary info needed for it to fetch the file and read it. Please let me know what you guys think? Or if this functionality already exists and I simply missed it?

abellgithub commented 1 month ago

A header and query option could be added. The rest of the support is already there.

trns1997 commented 1 month ago

A header and query option could be added. The rest of the support is already there.

Alrighty, I'll look into it, any tips on where I should look for inspiration, I was thinking of basing myself off the EptReaders class and go from there?

abellgithub commented 1 month ago

There's almost nothing to it. Just add the arguments and pass them to the getBinary call. Update the documentation.

abellgithub commented 1 month ago

I guess you also need to convert from JSON to string maps. You can copy CopcReader::setForwards()