4
84%
master: 84%

Ran 02 Oct 2020 02:09PM UTC

Files 0

Run time 0s

Badge

Embed ▾

Committed 02 Oct 2020 02:05PM UTC coverage: 0.0%. Remained the same

Job # ACTION=pylint OS=fedora OS_VERSION=31 PYTHON_VERSION=3 ENGINE=docker

Build Type

push

travis-ci

Committed by

MartinBasti

Commit Message

HttpStream: fix close method

`.close()` is called from `__del__` and because this happend during
garbage collecting nothing is guaranted [1] and it results into following
traceback printed directly to stderr because it happened in GC context:
```
Traceback (most recent call last):
  File "/home/mbasti/repos-upstream/osbs-client/osbs/http.py", line 221, in __del__
    self.close()
  File "/home/mbasti/repos-upstream/osbs-client/osbs/http.py", line 214, in close
    if not self.closed:
AttributeError: 'HttpStream' object has no attribute 'closed'
```

Using getattr instead to avoid traceback.

TLDR: use context managers if you want to be sure that connection is closed
properly.

[1] https://docs.python.org/3/reference/datamodel.html#object.__del__

Signed-off-by: Martin Bašti <mbasti@redhat.com>

Run Details

0 of 0 relevant lines covered (NaN%)

0.0 hits per line

containerbuildsystem / osbs-client / 5172 / 4
84%
master: 84%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 5172.4 (ACTION=pylint OS=fedora OS_VERSION=31 PYTHON_VERSION=3 ENGINE=docker)

containerbuildsystem / osbs-client / 5172 / 4 84% master: 84%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 5172.4 (ACTION=pylint OS=fedora OS_VERSION=31 PYTHON_VERSION=3 ENGINE=docker)

containerbuildsystem / osbs-client / 5172 / 4
84%
master: 84%

README BADGES
x