return in finally can swallow exceptions

[iritkatriel]

In

MerossIot/meross_iot/http_api.py

Line 372 in 58de4e0

return jsondata.get("data")

there is a return statement in a finally block, which would swallow any in-flight exception.

This means that if an exception other than ValueError(includingBaseException such as KeyboardInterrupt) is raised from the try body, or any exception is raised from an except: clause, it will not propagate on as expected.

See also https://docs.python.org/3/tutorial/errors.html#defining-clean-up-actions.

[donations-bot-prod]

Donation bot here. Supporters can mark their interest in this issue by adding a comment
starting with "+1" (without quotes) or adding the "#supporter_sponsor" keyword
(without quotes) in their comments.

You can become a donor by subscribing as Sponsor from the GitHub sponsorship page or from the BuyMeACoffee page.

Issues marked by donors will be labeled accordingly and handled with special care

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.