wrap: Use OSError instead of URLError for exception handling

URLError is a subclass of OSError and intermittent server errors can
manifest as OSError while reading instead of a URLError while
establishing a connection, which will cause the fallback url to be
ignored:

```
Looking for a fallback subproject for the dependency gudev-1.0
Downloading libgudev source from https://gitlab.gnome.org/GNOME/libgudev/-/archive/238/libgudev-238.tar.bz2
HTTP Error 404: Not Found
WARNING: failed to download with error: could not get https://gitlab.gnome.org/GNOME/libgudev/-/archive/238/libgudev-238.tar.bz2 is the internet available?. Trying after a delay...
HTTP Error 404: Not Found
WARNING: failed to download with error: could not get https://gitlab.gnome.org/GNOME/libgudev/-/archive/238/libgudev-238.tar.bz2 is the internet available?. Trying after a delay...
HTTP Error 404: Not Found
WARNING: failed to download with error: could not get https://gitlab.gnome.org/GNOME/libgudev/-/archive/238/libgudev-238.tar.bz2 is the internet available?. Trying after a delay...
WARNING: failed to download with error: The read operation timed out. Trying after a delay...
WARNING: failed to download with error: The read operation timed out. Trying after a delay...
ERROR: Unhandled python OSError. This is probably not a Meson bug, but an issue with your build environment.
```
pull/13474/head
Nirbheek Chauhan 4 months ago committed by Eli Schwartz
parent a544c750b1
commit 8a202de6ec
  1. 8
      mesonbuild/wrap/wrap.py

@ -76,9 +76,9 @@ def open_wrapdburl(urlstring: str, allow_insecure: bool = False, have_opt: bool
if has_ssl: if has_ssl:
try: try:
return T.cast('http.client.HTTPResponse', urllib.request.urlopen(urllib.parse.urlunparse(url), timeout=REQ_TIMEOUT)) return T.cast('http.client.HTTPResponse', urllib.request.urlopen(urllib.parse.urlunparse(url), timeout=REQ_TIMEOUT))
except urllib.error.URLError as excp: except OSError as excp:
msg = f'WrapDB connection failed to {urlstring} with error {excp}.' msg = f'WrapDB connection failed to {urlstring} with error {excp}.'
if isinstance(excp.reason, ssl.SSLCertVerificationError): if isinstance(excp, urllib.error.URLError) and isinstance(excp.reason, ssl.SSLCertVerificationError):
if allow_insecure: if allow_insecure:
mlog.warning(f'{msg}\n\n Proceeding without authentication.') mlog.warning(f'{msg}\n\n Proceeding without authentication.')
else: else:
@ -95,7 +95,7 @@ def open_wrapdburl(urlstring: str, allow_insecure: bool = False, have_opt: bool
nossl_url = url._replace(scheme='http') nossl_url = url._replace(scheme='http')
try: try:
return T.cast('http.client.HTTPResponse', urllib.request.urlopen(urllib.parse.urlunparse(nossl_url), timeout=REQ_TIMEOUT)) return T.cast('http.client.HTTPResponse', urllib.request.urlopen(urllib.parse.urlunparse(nossl_url), timeout=REQ_TIMEOUT))
except urllib.error.URLError as excp: except OSError as excp:
raise WrapException(f'WrapDB connection failed to {urlstring} with error {excp}') raise WrapException(f'WrapDB connection failed to {urlstring} with error {excp}')
def get_releases_data(allow_insecure: bool) -> bytes: def get_releases_data(allow_insecure: bool) -> bytes:
@ -704,7 +704,7 @@ class Resolver:
try: try:
req = urllib.request.Request(urlstring, headers=headers) req = urllib.request.Request(urlstring, headers=headers)
resp = urllib.request.urlopen(req, timeout=REQ_TIMEOUT) resp = urllib.request.urlopen(req, timeout=REQ_TIMEOUT)
except urllib.error.URLError as e: except OSError as e:
mlog.log(str(e)) mlog.log(str(e))
raise WrapException(f'could not get {urlstring} is the internet available?') raise WrapException(f'could not get {urlstring} is the internet available?')
with contextlib.closing(resp) as resp, tmpfile as tmpfile: with contextlib.closing(resp) as resp, tmpfile as tmpfile:

Loading…
Cancel
Save