HTTP endpoints: readings deleted on non-200 status code

[tbbuck]

Hi!

The documentation says that readings will only be deleted if an HTTP endpoint replies with a 200 status code.

My Python's a bit rusty, should line 34 be indented? Looks like readings are deleted no matter what status code is returned :-o

enviro/enviro/destinations/http.py

Lines 30 to 34 in 75ec515

	if result.status_code != 200:
	logging.error(f" - failed to upload '{cache_file}' ({result.status_code} {result.reason})", cache_file)
	else:
	logging.info(f" - uploaded {cache_file}")
	os.remove(f"uploads/{cache_file}")

(keep up the good work, loving my Enviro!)

EDIT: aha, looks like this pull request fixes things, including possibly a second issue I was investigating (requests fail after sending 3 readings): #15

[lowfatcode]

This has been fixed - thanks for pointing it out! cafed06