Follow HTML standards for query string List

[vincentsarago]

ref: #347, #262 (comment)

Problem description

Historically we choose to allow passing list values (bidx, bands, assets) as comma delimited value encoded in string (e.g bidx=1,2,3), which is then decoded to list

titiler/src/titiler/core/titiler/core/dependencies.py

Lines 89 to 101 in 1879706

	class BidxParams(DefaultDependency):
	"""Band Indexes parameters."""
	bidx: Optional[str] = Query(
	None, title="Band indexes", description="comma (',') delimited band indexes",
	)
	def __post_init__(self):
	"""Post Init."""
	if self.bidx is not None:
	self.kwargs["indexes"] = tuple(
	int(s) for s in re.findall(r"\d+", self.bidx)
	)

This works fine BUT is not standard.

Breaking change

this will be a breaking change but I feel it's better to follow standard.

[vincentsarago]

Side note:

Removing __post_init__ operation to parse and add values to kwargs would also allow removing the use of deps.kwargs (which is a hack in order to pass correct arguments to rio-tiler methods).

if dependency defines correct options we could then do something like:

@dataclass
class DefaultDependency:
    """Dataclass with dict unpacking"""

    def keys(self):
        return self.__dict__.keys()
    
    def __getitem__(self, key):
        return self.__dict__[key]


@dataclass
class BidxParams(DefaultDependency):
    """Band Indexes parameters."""

    indexes: Optional[List[str]] = Query(
        None, title="Band indexes", description="band indexes",
    )


def tile(
    z: int = Path(..., ge=0, le=30, description="Mercator tiles's zoom level"),
    x: int = Path(..., description="Mercator tiles's column"),
    y: int = Path(..., description="Mercator tiles's row"),
    src_path: DatasetPathParams = Depends(),    
    layer_params: BidxParams = Depends(),
)
    with COGReader( src_path) as src_dst:
        data = src_dst.tile(
            x,
            y,
            z,
            **layer_params,  # previously **layer_params.kwargs
        )