Revert "DRIVERS-3019 Add ruff linter and apply fixes" (#539)

Author: blink1073

.evergreen/auth_aws/aws_tester.py

@@ -1,16 +1,17 @@
+#!/usr/bin/env python3
 """
 Script for testing MONGDOB-AWS authentication.
 """
 import argparse
-import json
 import os
-import subprocess
+import json
 import sys
+import subprocess
 from functools import partial
-from urllib.parse import quote_plus
 
 from pymongo import MongoClient
 from pymongo.errors import OperationFailure
+from urllib.parse import quote_plus
 
 HERE = os.path.abspath(os.path.dirname(__file__))
 
@@ -19,10 +20,10 @@ def join(*parts):
 
 
 sys.path.insert(0, join(HERE, 'lib'))
-from aws_assign_instance_profile import _assign_instance_policy
+from util import get_key as _get_key
 from aws_assume_role import _assume_role
 from aws_assume_web_role import _assume_role_with_web_identity
-from util import get_key as _get_key
+from aws_assign_instance_profile import _assign_instance_policy
 
 ASSUMED_ROLE = "arn:aws:sts::557821124784:assumed-role/authtest_user_assume_role/*"
 ASSUMED_WEB_ROLE = "arn:aws:sts::857654397073:assumed-role/webIdentityTestRole/*"
@@ -43,7 +44,7 @@ def join(*parts):
 def run(args, env):
     """Run a python command in a subprocess."""
     env.update(os.environ.copy())
-    return subprocess.run([sys.executable, *args], env=env, check=False).returncode
+    return subprocess.run([sys.executable] + args, env=env).returncode
 
 
 def create_user(user, kwargs):

.evergreen/auth_aws/lib/aws_assign_instance_profile.py

@@ -1,18 +1,20 @@
+#!/usr/bin/env python3
 """
 Script for assign an instance policy to the current machine.
 """
 
 import argparse
-import json
+import urllib.request
 import logging
+import json
 import os
 import sys
 import time
-import urllib.request
 from functools import partial
 
 import boto3
 import botocore
+
 from util import get_key as _get_key
 
 sys.path.insert(1, os.path.join(sys.path[0], '..'))
@@ -39,7 +41,7 @@ def _has_instance_profile():
     try:
         url = base_url + iam_role
         print("Reading: " + url)
-        urllib.request.urlopen(url)
+        req = urllib.request.urlopen(url)
         print("Assigned " + iam_role)
     except urllib.error.HTTPError as e:
         print(e)

.evergreen/auth_aws/lib/aws_assume_role.py

@@ -1,10 +1,11 @@
+#!/usr/bin/env python3
 """
 Script for assuming an aws role.
 """
 
 import argparse
-import logging
 import uuid
+import logging
 
 import boto3
 

.evergreen/auth_aws/lib/aws_assume_web_role.py

@@ -1,11 +1,12 @@
+#!/usr/bin/env python3
 """
 Script for assuming an aws role using AssumeRoleWithWebIdentity.
 """
 
 import argparse
-import logging
 import os
 import uuid
+import logging
 
 import boto3
 

.evergreen/auth_aws/lib/aws_handle_oidc_creds.py

@@ -1,3 +1,4 @@
+#!/usr/bin/env python3
 """
 Script for handling OIDC credentials.
 """
@@ -14,6 +15,7 @@
 from pyop.userinfo import Userinfo
 
 
+
 class CustomSubjectIdentifierFactory(HashBasedSubjectIdentifierFactory):
     """
     Implements a hash based algorithm for creating a pairwise subject identifier.
@@ -31,7 +33,7 @@ def create_pairwise_identifier(self, user_id, sector_identifier):
 
 
 def get_default_config():
-    return {
+    config = {
         "issuer": os.getenv('IDP_ISSUER', ''),
         "jwks_uri": os.getenv('IDP_JWKS_URI', ''),
         'rsa_key': os.getenv('IDP_RSA_KEY', ''),
@@ -40,6 +42,7 @@ def get_default_config():
         'username': os.getenv("IDP_USERNAME", 'test_user'),
         'token_file': os.getenv('AWS_WEB_IDENTITY_TOKEN_FILE')
     }
+    return config
 
 
 def get_provider(config=None, expires=None):

.evergreen/auth_aws/lib/aws_unassign_instance_profile.py

@@ -1,13 +1,14 @@
+#!/usr/bin/env python3
 """
 Script for unassigning an instance policy from the current machine.
 """
 
 import argparse
+import urllib.error
+import urllib.request
 import logging
 import sys
 import time
-import urllib.error
-import urllib.request
 
 import boto3
 import botocore
@@ -31,7 +32,7 @@ def _has_instance_profile():
     try:
         url = base_url + iam_role
         print("Reading: " + url)
-        urllib.request.urlopen(url)
+        req = urllib.request.urlopen(url)
     except urllib.error.HTTPError as e:
         print(e)
         if e.code == 404:

.evergreen/auth_aws/lib/container_tester.py

@@ -1,3 +1,4 @@
+#!/usr/bin/env python3
 """
 Script for testing mongodb in containers.
 
@@ -40,7 +41,7 @@
 
 def _run_process(params, cwd=None):
     LOGGER.info("RUNNING COMMAND: %s", params)
-    ret = subprocess.run(params, cwd=cwd, check=False)
+    ret = subprocess.run(params, cwd=cwd)
     return ret.returncode
 
 def _userandhostandport(endpoint):
@@ -136,7 +137,7 @@ def remote_ps_container(cluster):
         assert private_ip_address
 
         eni = ec2_client.describe_network_interfaces(NetworkInterfaceIds=enis)
-        public_ip = next(iter(n["Association"]["PublicIp"] for n in eni["NetworkInterfaces"]))
+        public_ip = [n["Association"]["PublicIp"] for n in eni["NetworkInterfaces"]][0]
 
         for container in task['containers']:
             taskArn = container['taskArn']
@@ -145,7 +146,7 @@ def remote_ps_container(cluster):
             task_id = task_id + "/" + name
             lastStatus = container['lastStatus']
 
-        print(f"{task_id:<43}{lastStatus:<9}{public_ip:<25}{private_ip_address:<25}{taskDefinition_short:<16}")
+        print("{:<43}{:<9}{:<25}{:<25}{:<16}".format(task_id, lastStatus, public_ip, private_ip_address, taskDefinition_short ))
 
 def _remote_create_container_args(args):
     remote_create_container(args.cluster, args.task_definition, args.service, args.subnets, args.security_group)
@@ -246,7 +247,7 @@ def remote_get_public_endpoint_str(cluster, service_name):
         assert enis
 
         eni = ec2_client.describe_network_interfaces(NetworkInterfaceIds=enis)
-        public_ip = next(iter(n["Association"]["PublicIp"] for n in eni["NetworkInterfaces"]))
+        public_ip = [n["Association"]["PublicIp"] for n in eni["NetworkInterfaces"]][0]
         break
 
     return f"root@{public_ip}:22"

.evergreen/auth_aws/lib/util.py

@@ -1,4 +1,5 @@
 def get_key(key: str, uppercase: bool) -> str:
     if uppercase:
         return key.upper()
-    return key
+    else:
+        return key

.evergreen/auth_oidc/azure/handle_secrets.py

@@ -2,8 +2,8 @@
 import os
 from base64 import b64decode
 
-from azure.identity import DefaultAzureCredential
 from azure.keyvault.secrets import SecretClient
+from azure.identity import DefaultAzureCredential
 
 
 def main():

.evergreen/auth_oidc/azure/remote-scripts/test.py

@@ -1,8 +1,7 @@
-import json
-import os
-from urllib.request import Request, urlopen
-
 from pymongo import MongoClient
+import os
+import json
+from urllib.request import urlopen, Request
 from pymongo.auth_oidc import OIDCCallback, OIDCCallbackContext, OIDCCallbackResult
 
 app_id = os.environ['AZUREOIDC_APPID']
@@ -23,16 +22,16 @@ def fetch(self, context: OIDCCallbackContext) -> OIDCCallbackResult:
                 body = response.read().decode('utf8')
         except Exception as e:
             msg = "Failed to acquire IMDS access token: %s" % e
-            raise ValueError(msg) from e
+            raise ValueError(msg)
 
         if status != 200:
             print(body)
             msg = "Failed to acquire IMDS access token."
             raise ValueError(msg)
         try:
             data = json.loads(body)
-        except Exception as e:
-            raise ValueError("Azure IMDS response must be in JSON format.") from e
+        except Exception:
+            raise ValueError("Azure IMDS response must be in JSON format.")
 
         for key in ["access_token", "expires_in"]:
             if not data.get(key):