uktrade / public-data-api

The source for the Department for International Trade's Public Data API
https://data.api.trade.gov.uk/
MIT License
5 stars 0 forks source link

build(deps): upgrade dependencies #189

Closed michalc closed 8 months ago

michalc commented 8 months ago

... that have reported vulnerabilities

The changes in the tests, closing http connections, are not for completely known reasons - without this the tests time out waiting the server under test to exit. It's supspected that without this Python requests keeps connections open, which in turn makes the Gevent web server keep its greenlets running, that makes them wait forever when attempting to exit the sever gracefully.

Why this happened now and not before, it's not known.

However, this change should be a fairly safe thing to do because in GOV.UK PaaS the last hop of the HTTP chains to the server through the various layers are not kept alive - a new TCP connection is made on each HTTP request. Thus the tests are now more like the PaaS environment than they were before.