tile cache probably never enabled, likely discarded if it was, and abort signal passed to options argument

[jcphill]

The GeoTIFFImage constructor is passed this.cache by GeoTIFF.getImage(),
which is set in the GeoTIFF constructor based its options argument,
which is passed through by fromSource(),
but which is not passed through by fromUrl(), fromCustomClient(), or fromUrls()
as they pass their options argument only to makeRemoteSource() or makeCustomSource()
and actually pass their signal argument as the second (options) argument of fromSource()
so the signal argument isn't doing anything either.

It is possible to set cache=true on a GeoTIFF object after construction.
But GeoTIFF.getImage() creates a new GeoTIFFImage object with an empty tile cache on every call
and GeoTIFF.readRasters() doesn't re-use images
so the cached tiles are are never accessed anyway.

[sguimmara]

The fix should be pretty simple:

export async function fromCustomClient(client, options = {}, signal) {
-  return GeoTIFF.fromSource(makeCustomSource(client, options), signal);
+  return GeoTIFF.fromSource(makeCustomSource(client, options), options, signal);
}

[sguimmara]

I think it is a duplicate of #330