1
0
mirror of https://github.com/openSUSE/osc.git synced 2024-11-11 07:06:16 +01:00

Merge pull request #1249 from dmach/fix-cookiejar-load-error

connection: Save cookiejar only when there's a Set-Cookie header in the response
This commit is contained in:
Daniel Mach 2023-02-02 14:11:58 +01:00 committed by GitHub
commit 34ad8f3dd8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -451,7 +451,10 @@ class CookieJarAuthHandler(AuthHandlerBase):
pass
jar = http.cookiejar.LWPCookieJar(self.cookiejar_path)
if os.path.isfile(self.cookiejar_path):
try:
jar.load()
except http.cookiejar.LoadError:
pass
self.COOKIEJARS[self.cookiejar_path] = jar
return jar
@ -484,6 +487,7 @@ class CookieJarAuthHandler(AuthHandlerBase):
return False
def process_response(self, url, request_headers, response):
if response.headers.get_all("set-cookie", None):
self._cookiejar.extract_cookies(response, MockRequest(url, response.headers))
self._cookiejar.save()
self._unlock()