search

ioos / erddapy

Python interface for ERDDAP
https://ioos.github.io/erddapy/
BSD 3-Clause "New" or "Revised" License
77 stars 30 forks source link

oddily ncCF datasets cannot be quoted #361

Closed ocefpaf closed 1 month ago

ocefpaf commented 1 month ago

I'm starting to wonder if we do need to quote all URLs. The behavior doesn't seem to be consistent as Bob told me. Maybe we should revert all this and quote only the rare cases where " is present. Anyway, let's give this one last go.

MathewBiddle commented 1 month ago

Is this something to push upstream to ERDDAP proper?

ocefpaf commented 1 month ago

Is this something to push upstream to ERDDAP proper?

Not sure but in version <2.23 we could pass unquoted URLs, no problem. Now, for >=2.23, we need to quote sometimes, kind of odd but I'm not an expert on URLs standards. I do recall Bob warning me that we should always quote. However, that breaks a few corner cases, so far the search URLs and the ncCF. Hopefully that is it...