From dec181727cda490bdd128211362dacac3ea8641f Mon Sep 17 00:00:00 2001 From: jlarriba Date: Mon, 8 Jul 2024 12:42:26 +0200 Subject: [PATCH] If-else with same path in both directions An analysis of the code with a static analysis tool (OpenScanHub) revealed that there is an if-else block that had identical 'then' and 'else' branches. It seems clear that the if-else block is no longer needed, so this patch removes it. Change-Id: I5152c5d58d0db263045cb95d47d594c518bbc8da --- ceilometermiddleware/swift.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/ceilometermiddleware/swift.py b/ceilometermiddleware/swift.py index 364611b..9e0082c 100644 --- a/ceilometermiddleware/swift.py +++ b/ceilometermiddleware/swift.py @@ -317,10 +317,7 @@ class Swift(object): for header in env: if header.startswith('HTTP_') and env[header]: key = header[5:] - if isinstance(env[header], str): - headers[key] = str(env[header]) - else: - headers[key] = str(env[header]) + headers[key] = str(env[header]) try: container = obj = None