Need special case for SVG files in network.check_reponse()?

[caternuson]

This guide:
https://learn.adafruit.com/pyportal-discord-online-count/overview
uses code:
https://github.com/adafruit/Adafruit_Learning_System_Guides/blob/main/PyPortal_Discord/code.py
that parses the XML found in a downloaded SVG using regex. That requires a return type of CONTENT_TEXT from check_response() for the PyPortal library to work as expected:
https://github.com/adafruit/Adafruit_CircuitPython_PyPortal/blob/8031c95bbb209a2e4ad5bec7628fd02a4614da40/adafruit_pyportal/__init__.py#L310

However, this library will end up returning CONTENT_IMAGE since the fetch headers will contain content-type: image/svg_html.

Adafruit_CircuitPython_PortalBase/adafruit_portalbase/network.py

Lines 493 to 500 in 6a94293

	def _detect_content_type(self, headers):
	if "content-type" in headers:
	if "image/" in headers["content-type"]:
	return CONTENT_IMAGE
	for json_type in self._json_types:
	if json_type in headers["content-type"]:
	return CONTENT_JSON
	return CONTENT_TEXT

That breaks the code for the above guide.

Opening issue here, but could maybe be fixed in one of the higher level libraries, like PyPortal?

[makermelissa]

I'd suggest adding an optional parameter to this library to force a particular content type which would make it skip the autodetecting.

[dearmash]

I second the suggestion of forcing a content type. I have a similar issue noted in https://forums.adafruit.com/viewtopic.php?f=59&t=187225 where a json response from a server is identified as text/plain and can't be correctly parsed.

My solution was to force json if the type was text/plain if the json_path is specified for parsing the response. It makes just as much sense, and can be solved the same way, by specifying a content_type_override of some sort.

[FoamyGuy]

I am working on a fix for this, will PR soon.