rucio 37.2.0__py3-none-any.whl → 37.3.0__py3-none-any.whl

rucio/web/rest/flaskapi/v1/main.py

@@ -15,15 +15,19 @@
 
 import importlib
 import logging
+from typing import TYPE_CHECKING
 
 from flask import Flask
 
-from rucio.common.config import config_get
+from rucio.common.config import config_get_list
 from rucio.common.exception import ConfigurationError
 from rucio.common.logging import setup_logging
 from rucio.web.rest.flaskapi.v1.common import CORSMiddleware
 
-DEFAULT_ENDPOINTS = [
+if TYPE_CHECKING:
+    from collections.abc import Iterable
+
+DEFAULT_ENDPOINTS = {
     'accountlimits',
     'accounts',
     'auth',
@@ -46,10 +50,10 @@ DEFAULT_ENDPOINTS = [
     'rules',
     'scopes',
     'subscriptions',
-]
+}
 
 
-def apply_endpoints(app, modules):
+def apply_endpoints(app: Flask, modules: "Iterable[str]") -> None:
     for blueprint_module in modules:
         # Legacy patch - TODO Remove in 38.0.0
         if blueprint_module == "meta":
@@ -71,15 +75,18 @@ def apply_endpoints(app, modules):
         else:
             raise ConfigurationError(f'"{blueprint_module}" from the endpoints configuration value did not have a blueprint')
 
+endpoints = set(config_get_list('api', 'endpoints', raise_exception=False, default=[]))
+endpoints_add = set(config_get_list('api', 'endpoints_add', raise_exception=False, default=[]))
+endpoints_remove = set(config_get_list('api', 'endpoints_remove', raise_exception=False, default=[]))
+
+if endpoints and (endpoints_add or endpoints_remove):
+    raise ConfigurationError("Endpoints cannot be set in both 'endpoints' and 'endpoints_add'/'endpoints_remove'")
 
-try:
-    endpoints = config_get('api', 'endpoints', raise_exception=False, default='')
-    endpoints = list(filter(bool, map(str.strip, endpoints.split(sep=','))))
-except RuntimeError:
-    endpoints = None
+if endpoints_add.intersection(endpoints_remove):
+    raise ConfigurationError("Endpoints cannot be in both 'endpoints_add' and 'endpoints_remove'")
 
 if not endpoints:
-    endpoints = DEFAULT_ENDPOINTS
+    endpoints = DEFAULT_ENDPOINTS - endpoints_remove | endpoints_add
 
 application = Flask(__name__)
 application.wsgi_app = CORSMiddleware(application.wsgi_app)

rucio/web/rest/flaskapi/v1/replicas.py

@@ -226,7 +226,7 @@ class Replicas(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            scope, name = parse_scope_name(scope_name, request.environ.get('vo'))
+            scope, name = parse_scope_name(scope_name, request.environ['vo'])
         except ValueError as error:
             return generate_http_error_flask(400, error)
 
@@ -264,7 +264,7 @@ class Replicas(ErrorHandlingMethodView):
                     rfile['pfns'] = dict(_sorted_with_priorities(rfile['pfns'], replicas, limit=limit))
                     yield rfile
 
-            rfiles = _list_and_sort_replicas(vo=request.environ.get('vo'))
+            rfiles = _list_and_sort_replicas(vo=request.environ['vo'])
             if metalink:
                 response_generator = _generate_metalink_response(rfiles, 'atlas', detailed_url=False)
             else:
@@ -359,8 +359,8 @@ class Replicas(ErrorHandlingMethodView):
             add_replicas(
                 rse=rse,
                 files=files,
-                issuer=request.environ.get('issuer'),
-                vo=request.environ.get('vo'),
+                issuer=request.environ['issuer'],
+                vo=request.environ['vo'],
                 ignore_availability=param_get(parameters, 'ignore_availability', default=False),
             )
         except InvalidPath as error:
@@ -432,7 +432,7 @@ class Replicas(ErrorHandlingMethodView):
         files = param_get(parameters, 'files')
 
         try:
-            update_replicas_states(rse=rse, files=files, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            update_replicas_states(rse=rse, files=files, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
 
@@ -486,8 +486,8 @@ class Replicas(ErrorHandlingMethodView):
             delete_replicas(
                 rse=rse,
                 files=files,
-                issuer=request.environ.get('issuer'),
-                vo=request.environ.get('vo'),
+                issuer=request.environ['issuer'],
+                vo=request.environ['vo'],
                 ignore_availability=param_get(parameters, 'ignore_availability', default=False),
             )
         except AccessDenied as error:
@@ -746,8 +746,8 @@ class ListReplicas(ErrorHandlingMethodView):
                     yield rfile
 
             rfiles = _list_and_sort_replicas(request_id=request.environ.get('request_id'),
-                                             issuer=request.environ.get('issuer'),
-                                             vo=request.environ.get('vo'))
+                                             issuer=request.environ['issuer'],
+                                             vo=request.environ['vo'])
             if metalink:
                 policy_schema = config_get('policy', 'schema', raise_exception=False, default='generic')
                 response_generator = _generate_metalink_response(rfiles, policy_schema)
@@ -822,7 +822,7 @@ class ReplicasDIDs(ErrorHandlingMethodView):
                 for pfn in get_did_from_pfns(pfns, rse, vo=vo):
                     yield dumps(pfn) + '\n'
 
-            return try_stream(generate(vo=request.environ.get('vo')))
+            return try_stream(generate(vo=request.environ['vo']))
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
 
@@ -882,7 +882,7 @@ class BadReplicas(ErrorHandlingMethodView):
 
         try:
             not_declared_files = declare_bad_file_replicas(replicas, reason=reason,
-                                                           issuer=request.environ.get('issuer'), vo=request.environ.get('vo'),
+                                                           issuer=request.environ['issuer'], vo=request.environ['vo'],
                                                            force=force)
             return not_declared_files, 201
         except AccessDenied as error:
@@ -1002,7 +1002,7 @@ class SuspiciousReplicas(ErrorHandlingMethodView):
         reason = param_get(parameters, 'reason', default=None)
 
         try:
-            not_declared_files = declare_suspicious_file_replicas(pfns=pfns, reason=reason, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            not_declared_files = declare_suspicious_file_replicas(pfns=pfns, reason=reason, issuer=request.environ['issuer'], vo=request.environ['vo'])
             return not_declared_files, 201
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
@@ -1081,7 +1081,7 @@ class SuspiciousReplicas(ErrorHandlingMethodView):
             if 'nattempts' in params:
                 nattempts = int(params['nattempts'][0])
 
-        result = get_suspicious_files(rse_expression=rse_expression, younger_than=younger_than, nattempts=nattempts, vo=request.environ.get('vo'))
+        result = get_suspicious_files(rse_expression=rse_expression, younger_than=younger_than, nattempts=nattempts, vo=request.environ['vo'])
         return Response(render_json(result), 200, content_type='application/json')
 
 
@@ -1180,7 +1180,8 @@ class BadReplicasStates(ErrorHandlingMethodView):
           406:
             description: Not acceptable
         """
-        state, rse, younger_than, older_than, limit, list_pfns = None, None, None, None, None, None
+        list_pfns = False
+        state, rse, younger_than, older_than, limit = None, None, None, None, None
         if request.query_string:
             query_string = request.query_string.decode(encoding='utf-8')
             try:
@@ -1208,7 +1209,7 @@ class BadReplicasStates(ErrorHandlingMethodView):
                                                 vo=vo):
                 yield dumps(row, cls=APIEncoder) + '\n'
 
-        return try_stream(generate(vo=request.environ.get('vo')))
+        return try_stream(generate(vo=request.environ['vo']))
 
 
 class BadReplicasSummary(ErrorHandlingMethodView):
@@ -1287,7 +1288,7 @@ class BadReplicasSummary(ErrorHandlingMethodView):
                                                 to_date=to_date, vo=vo):
                 yield dumps(row, cls=APIEncoder) + '\n'
 
-        return try_stream(generate(vo=request.environ.get('vo')))
+        return try_stream(generate(vo=request.environ['vo']))
 
 
 class DatasetReplicas(ErrorHandlingMethodView):
@@ -1370,7 +1371,7 @@ class DatasetReplicas(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            scope, name = parse_scope_name(scope_name, request.environ.get('vo'))
+            scope, name = parse_scope_name(scope_name, request.environ['vo'])
 
             def generate(_deep, vo):
                 for row in list_dataset_replicas(scope=scope, name=name, deep=_deep, vo=vo):
@@ -1378,7 +1379,7 @@ class DatasetReplicas(ErrorHandlingMethodView):
 
             deep = request.args.get('deep', default=False)
 
-            return try_stream(generate(_deep=deep, vo=request.environ.get('vo')))
+            return try_stream(generate(_deep=deep, vo=request.environ['vo']))
         except ValueError as error:
             return generate_http_error_flask(400, error)
 
@@ -1483,7 +1484,7 @@ class DatasetReplicasBulk(ErrorHandlingMethodView):
                 for row in list_dataset_replicas_bulk(dids=dids, vo=vo):
                     yield dumps(row, cls=APIEncoder) + '\n'
 
-            return try_stream(generate(vo=request.environ.get('vo')))
+            return try_stream(generate(vo=request.environ['vo']))
         except InvalidObject as error:
             return generate_http_error_flask(400, error, f'Cannot validate DIDs: {error}')
 
@@ -1520,7 +1521,7 @@ class DatasetReplicasVP(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            scope, name = parse_scope_name(scope_name, request.environ.get('vo'))
+            scope, name = parse_scope_name(scope_name, request.environ['vo'])
 
             def generate(_deep, vo):
                 for row in list_dataset_replicas_vp(scope=scope, name=name, deep=_deep, vo=vo):
@@ -1528,7 +1529,7 @@ class DatasetReplicasVP(ErrorHandlingMethodView):
 
             deep = request.args.get('deep', default=False)
 
-            return try_stream(generate(_deep=deep, vo=request.environ.get('vo')))
+            return try_stream(generate(_deep=deep, vo=request.environ['vo']))
         except ValueError as error:
             return generate_http_error_flask(400, error)
 
@@ -1610,7 +1611,7 @@ class ReplicasRSE(ErrorHandlingMethodView):
             for row in list_datasets_per_rse(rse=rse, vo=vo):
                 yield dumps(row, cls=APIEncoder) + '\n'
 
-        return try_stream(generate(vo=request.environ.get('vo')))
+        return try_stream(generate(vo=request.environ['vo']))
 
 
 class BadDIDs(ErrorHandlingMethodView):
@@ -1676,11 +1677,11 @@ class BadDIDs(ErrorHandlingMethodView):
             not_declared_files = add_bad_dids(
                 dids=param_get(parameters, 'dids', default=[]),
                 rse=param_get(parameters, 'rse', default=None),
-                issuer=request.environ.get('issuer'),
+                issuer=request.environ['issuer'],
                 state=BadFilesStatus.BAD,
                 reason=param_get(parameters, 'reason', default=None),
                 expires_at=expires_at,
-                vo=request.environ.get('vo'),
+                vo=request.environ['vo'],
             )
         except (ValueError, InvalidType) as error:
             return generate_http_error_flask(400, ValueError.__name__, error.args[0])
@@ -1745,11 +1746,11 @@ class BadPFNs(ErrorHandlingMethodView):
         try:
             add_bad_pfns(
                 pfns=param_get(parameters, 'pfns', default=[]),
-                issuer=request.environ.get('issuer'),
+                issuer=request.environ['issuer'],
                 state=param_get(parameters, 'state', default=None),
                 reason=param_get(parameters, 'reason', default=None),
                 expires_at=expires_at,
-                vo=request.environ.get('vo'),
+                vo=request.environ['vo'],
             )
         except (ValueError, InvalidType) as error:
             return generate_http_error_flask(400, ValueError.__name__, error.args[0])
@@ -1816,8 +1817,8 @@ class Tombstone(ErrorHandlingMethodView):
                     rse=replica['rse'],
                     scope=replica['scope'],
                     name=replica['name'],
-                    issuer=request.environ.get('issuer'),
-                    vo=request.environ.get('vo'),
+                    issuer=request.environ['issuer'],
+                    vo=request.environ['vo'],
                 )
         except ReplicaNotFound as error:
             return generate_http_error_flask(404, error)

rucio/web/rest/flaskapi/v1/rses.py

@@ -178,7 +178,7 @@ class RSEs(ErrorHandlingMethodView):
                     for rse in parse_rse_expression(expression, vo=vo):
                         yield render_json(rse=rse) + '\n'
 
-                return try_stream(generate(vo=request.environ.get('vo')))
+                return try_stream(generate(vo=request.environ['vo']))
             except (InvalidRSEExpression, InvalidObject) as error:
                 return generate_http_error_flask(400, error)
         else:
@@ -187,7 +187,7 @@ class RSEs(ErrorHandlingMethodView):
                     rse['availability'] = Availability(rse['availability_read'], rse['availability_write'], rse['availability_delete']).integer
                     yield render_json(**rse) + '\n'
 
-            return try_stream(generate(vo=request.environ.get('vo')))
+            return try_stream(generate(vo=request.environ['vo']))
 
 
 class RSE(ErrorHandlingMethodView):
@@ -316,8 +316,8 @@ class RSE(ErrorHandlingMethodView):
             for keyword in kwargs.keys():
                 kwargs[keyword] = param_get(parameters, keyword, default=kwargs[keyword])
 
-        kwargs['issuer'] = request.environ.get('issuer')
-        kwargs['vo'] = request.environ.get('vo')
+        kwargs['issuer'] = request.environ['issuer']
+        kwargs['vo'] = request.environ['vo']
         try:
             add_rse(rse, **kwargs)
         except InvalidObject as error:
@@ -408,8 +408,8 @@ class RSE(ErrorHandlingMethodView):
         """
         kwargs = {
             'parameters': json_parameters(optional=True),
-            'issuer': request.environ.get('issuer'),
-            'vo': request.environ.get('vo'),
+            'issuer': request.environ['issuer'],
+            'vo': request.environ['vo'],
         }
         try:
             update_rse(rse, **kwargs)
@@ -508,7 +508,7 @@ class RSE(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            rse = get_rse(rse=rse, vo=request.environ.get('vo'))
+            rse = get_rse(rse=rse, vo=request.environ['vo'])
             rse['availability'] = Availability(rse['availability_read'], rse['availability_write'], rse['availability_delete']).integer
             return Response(render_json(**rse), content_type="application/json")
         except RSENotFound as error:
@@ -537,7 +537,7 @@ class RSE(ErrorHandlingMethodView):
             description: RSE not found
         """
         try:
-            del_rse(rse=rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            del_rse(rse=rse, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except (RSENotFound, RSEOperationNotSupported, CounterNotFound) as error:
             return generate_http_error_flask(404, error)
         except AccessDenied as error:
@@ -600,7 +600,7 @@ class Attributes(ErrorHandlingMethodView):
         parameters = json_parameters()
         value = param_get(parameters, 'value')
         try:
-            add_rse_attribute(rse=rse, key=key, value=value, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            add_rse_attribute(rse=rse, key=key, value=value, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
         except Duplicate as error:
@@ -643,7 +643,7 @@ class Attributes(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            rse_attr = list_rse_attributes(rse, vo=request.environ.get('vo'))
+            rse_attr = list_rse_attributes(rse, vo=request.environ['vo'])
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
         except RSENotFound as error:
@@ -680,7 +680,7 @@ class Attributes(ErrorHandlingMethodView):
             description: RSE or RSE attribute not found
         """
         try:
-            del_rse_attribute(rse=rse, key=key, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            del_rse_attribute(rse=rse, key=key, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
         except (RSENotFound, RSEAttributeNotFound) as error:
@@ -831,7 +831,7 @@ class ProtocolList(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            p_list = get_rse_protocols(rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            p_list = get_rse_protocols(rse, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except (RSEOperationNotSupported, RSENotFound, RSEProtocolNotSupported, RSEProtocolDomainNotSupported) as error:
             return generate_http_error_flask(404, error)
 
@@ -909,7 +909,7 @@ class LFNS2PFNS(ErrorHandlingMethodView):
         operation = request.args.get('operation', default='write')
 
         try:
-            rse_settings = get_rse_protocols(rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            rse_settings = get_rse_protocols(rse, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except (RSENotFound, RSEProtocolNotSupported, RSEProtocolDomainNotSupported) as error:
             return generate_http_error_flask(404, error)
 
@@ -1009,7 +1009,7 @@ class Protocol(ErrorHandlingMethodView):
         parameters['scheme'] = scheme
 
         try:
-            add_protocol(rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'), data=parameters)
+            add_protocol(rse, issuer=request.environ['issuer'], vo=request.environ['vo'], data=parameters)
         except (RSENotFound, RSEProtocolDomainNotSupported) as error:
             return generate_http_error_flask(404, error)
         except AccessDenied as error:
@@ -1167,7 +1167,7 @@ class Protocol(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            p_list = get_rse_protocols(rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            p_list = get_rse_protocols(rse, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except (RSENotFound, RSEProtocolNotSupported, RSEProtocolDomainNotSupported) as error:
             return generate_http_error_flask(404, error)
 
@@ -1334,8 +1334,8 @@ class Protocol(ErrorHandlingMethodView):
         try:
             update_protocols(
                 rse,
-                issuer=request.environ.get('issuer'),
-                vo=request.environ.get('vo'),
+                issuer=request.environ['issuer'],
+                vo=request.environ['vo'],
                 scheme=scheme,
                 hostname=hostname,
                 port=int(port) if port else None,
@@ -1395,8 +1395,8 @@ class Protocol(ErrorHandlingMethodView):
         try:
             del_protocols(
                 rse,
-                issuer=request.environ.get('issuer'),
-                vo=request.environ.get('vo'),
+                issuer=request.environ['issuer'],
+                vo=request.environ['vo'],
                 scheme=scheme,
                 hostname=hostname,
                 port=int(port) if port else None
@@ -1485,10 +1485,10 @@ class Usage(ErrorHandlingMethodView):
 
             return try_stream(
                 generate(
-                    issuer=request.environ.get('issuer'),
+                    issuer=request.environ['issuer'],
                     source=request.args.get('source'),
                     per_account=per_account,
-                    vo=request.environ.get('vo'),
+                    vo=request.environ['vo'],
                 )
             )
         except RSENotFound as error:
@@ -1544,7 +1544,7 @@ class Usage(ErrorHandlingMethodView):
             kwargs[keyword] = param_get(parameters, keyword, default=kwargs[keyword])
 
         try:
-            set_rse_usage(rse=rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'), **kwargs)
+            set_rse_usage(rse=rse, issuer=request.environ['issuer'], vo=request.environ['vo'], **kwargs)
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
         except RSENotFound as error:
@@ -1615,7 +1615,7 @@ class UsageHistory(ErrorHandlingMethodView):
                 for usage in list_rse_usage_history(rse=rse, issuer=issuer, source=source, vo=vo):
                     yield render_json(**usage) + '\n'
 
-            return try_stream(generate(issuer=request.environ.get('issuer'), source=request.args.get('source'), vo=request.environ.get('vo')))
+            return try_stream(generate(issuer=request.environ['issuer'], source=request.args.get('source'), vo=request.environ['vo']))
         except RSENotFound as error:
             return generate_http_error_flask(404, error)
 
@@ -1658,7 +1658,7 @@ class Limits(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            limits = get_rse_limits(rse=rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            limits = get_rse_limits(rse=rse, issuer=request.environ['issuer'], vo=request.environ['vo'])
             return Response(render_json(**limits), content_type="application/json")
         except RSENotFound as error:
             return generate_http_error_flask(404, error)
@@ -1705,7 +1705,7 @@ class Limits(ErrorHandlingMethodView):
             kwargs[keyword] = param_get(parameters, keyword, default=kwargs[keyword])
 
         try:
-            set_rse_limits(rse=rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'), **kwargs)
+            set_rse_limits(rse=rse, issuer=request.environ['issuer'], vo=request.environ['vo'], **kwargs)
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
         except RSENotFound as error:
@@ -1752,7 +1752,7 @@ class Limits(ErrorHandlingMethodView):
         name = param_get(parameters, 'name')
 
         try:
-            delete_rse_limits(rse=rse, name=name, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            delete_rse_limits(rse=rse, name=name, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
         except RSENotFound as error:
@@ -1820,7 +1820,7 @@ class RSEAccountUsageLimit(ErrorHandlingMethodView):
                 for usage in get_rse_account_usage(rse=rse, vo=vo):
                     yield render_json(**usage) + '\n'
 
-            return try_stream(generate(vo=request.environ.get('vo')), content_type='application/json')
+            return try_stream(generate(vo=request.environ['vo']), content_type='application/json')
         except RSENotFound as error:
             return generate_http_error_flask(404, error)
 
@@ -1882,7 +1882,7 @@ class Distance(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            distance = get_distance(source=source, destination=destination, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            distance = get_distance(source=source, destination=destination, issuer=request.environ['issuer'], vo=request.environ['vo'])
             return Response(dumps(distance, cls=APIEncoder), content_type="application/json")
         except RSENotFound as error:
             return generate_http_error_flask(404, error)
@@ -1946,8 +1946,8 @@ class Distance(ErrorHandlingMethodView):
                 source=source,
                 destination=destination,
                 distance=distance,
-                issuer=request.environ.get('issuer'),
-                vo=request.environ.get('vo'),
+                issuer=request.environ['issuer'],
+                vo=request.environ['vo'],
             )
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
@@ -2017,8 +2017,8 @@ class Distance(ErrorHandlingMethodView):
                 source=source,
                 destination=destination,
                 distance=distance,
-                issuer=request.environ.get('issuer'),
-                vo=request.environ.get('vo'),
+                issuer=request.environ['issuer'],
+                vo=request.environ['vo'],
             )
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
@@ -2066,8 +2066,8 @@ class Distance(ErrorHandlingMethodView):
             delete_distance(
                 source=source,
                 destination=destination,
-                issuer=request.environ.get('issuer'),
-                vo=request.environ.get('vo')
+                issuer=request.environ['issuer'],
+                vo=request.environ['vo']
             )
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
@@ -2117,7 +2117,7 @@ class QoSPolicy(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            add_qos_policy(rse=rse, qos_policy=policy, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            add_qos_policy(rse=rse, qos_policy=policy, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except RSENotFound as error:
             return generate_http_error_flask(404, error)
 
@@ -2155,7 +2155,7 @@ class QoSPolicy(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            delete_qos_policy(rse=rse, qos_policy=policy, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            delete_qos_policy(rse=rse, qos_policy=policy, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except RSENotFound as error:
             return generate_http_error_flask(404, error)
 
@@ -2201,7 +2201,7 @@ class QoSPolicy(ErrorHandlingMethodView):
             description: Not acceptable
         """
         try:
-            qos_policies = list_qos_policies(rse=rse, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            qos_policies = list_qos_policies(rse=rse, issuer=request.environ['issuer'], vo=request.environ['vo'])
             return Response(dumps(qos_policies, cls=APIEncoder), content_type='application/json')
         except RSENotFound as error:
             return generate_http_error_flask(404, error)

rucio/web/rest/flaskapi/v1/rules.py

@@ -92,7 +92,7 @@ class Rule(ErrorHandlingMethodView):
             return generate_http_error_flask(501, "NotImplemented", exc_msg="estimate_ttc is not implemented!")
 
         try:
-            rule = get_replication_rule(rule_id, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            rule = get_replication_rule(rule_id, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except RuleNotFound as error:
             return generate_http_error_flask(404, error)
 
@@ -177,7 +177,7 @@ class Rule(ErrorHandlingMethodView):
         parameters = json_parameters()
         options: dict[str, Any] = param_get(parameters, 'options')
         try:
-            update_replication_rule(rule_id=rule_id, options=options, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            update_replication_rule(rule_id=rule_id, options=options, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except AccessDenied as error:
             return generate_http_error_flask(401, error)
         except (RuleNotFound, AccountNotFound) as error:
@@ -212,7 +212,7 @@ class Rule(ErrorHandlingMethodView):
         parameters = json_parameters()
         purge_replicas = param_get(parameters, 'purge_replicas', default=None)
         try:
-            delete_replication_rule(rule_id=rule_id, purge_replicas=purge_replicas, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+            delete_replication_rule(rule_id=rule_id, purge_replicas=purge_replicas, issuer=request.environ['issuer'], vo=request.environ['vo'])
         except (AccessDenied, UnsupportedOperation) as error:
             return generate_http_error_flask(401, error)
         except RuleNotFound as error:
@@ -250,7 +250,7 @@ class AllRule(ErrorHandlingMethodView):
                 for rule in list_replication_rules(filters=filters, vo=vo):
                     yield dumps(rule, cls=APIEncoder) + '\n'
 
-            return try_stream(generate(filters=dict(request.args.items(multi=False)), vo=request.environ.get('vo')))
+            return try_stream(generate(filters=dict(request.args.items(multi=False)), vo=request.environ['vo']))
         except RuleNotFound as error:
             return generate_http_error_flask(404, error)
 
@@ -397,8 +397,8 @@ class AllRule(ErrorHandlingMethodView):
                 priority=param_get(parameters, 'priority', default=3),
                 split_container=param_get(parameters, 'split_container', default=False),
                 meta=param_get(parameters, 'meta', default=None),
-                issuer=request.environ.get('issuer'),
-                vo=request.environ.get('vo'),
+                issuer=request.environ['issuer'],
+                vo=request.environ['vo'],
             )
         except (
             InvalidReplicationRule,
@@ -477,7 +477,7 @@ class ReplicaLocks(ErrorHandlingMethodView):
             for lock in get_replica_locks_for_rule_id(rule_id, vo=vo):
                 yield render_json(**lock) + '\n'
 
-        return try_stream(generate(vo=request.environ.get('vo')))
+        return try_stream(generate(vo=request.environ['vo']))
 
 
 class ReduceRule(ErrorHandlingMethodView):
@@ -531,8 +531,8 @@ class ReduceRule(ErrorHandlingMethodView):
             rule_ids = reduce_replication_rule(rule_id=rule_id,
                                                copies=copies,
                                                exclude_expression=exclude_expression,
-                                               issuer=request.environ.get('issuer'),
-                                               vo=request.environ.get('vo'))
+                                               issuer=request.environ['issuer'],
+                                               vo=request.environ['vo'])
         # TODO: Add all other error cases here
         except RuleReplaceFailed as error:
             return generate_http_error_flask(409, error)
@@ -612,8 +612,8 @@ class MoveRule(ErrorHandlingMethodView):
             rule_ids = move_replication_rule(rule_id=rule_id,
                                              rse_expression=rse_expression,
                                              override=override,
-                                             issuer=request.environ.get('issuer'),
-                                             vo=request.environ.get('vo'))
+                                             issuer=request.environ['issuer'],
+                                             vo=request.environ['vo'])
         except RuleReplaceFailed as error:
             return generate_http_error_flask(409, error)
         except RuleNotFound as error:
@@ -676,7 +676,7 @@ class RuleHistory(ErrorHandlingMethodView):
             for history in list_replication_rule_history(rule_id, issuer=issuer, vo=vo):
                 yield render_json(**history) + '\n'
 
-        return try_stream(generate(issuer=request.environ.get('issuer'), vo=request.environ.get('vo')))
+        return try_stream(generate(issuer=request.environ['issuer'], vo=request.environ['vo']))
 
 
 class RuleHistoryFull(ErrorHandlingMethodView):
@@ -740,13 +740,13 @@ class RuleHistoryFull(ErrorHandlingMethodView):
             description: Not acceptable.
         """
         try:
-            scope, name = parse_scope_name(scope_name, request.environ.get('vo'))
+            scope, name = parse_scope_name(scope_name, request.environ['vo'])
 
             def generate(vo):
                 for history in list_replication_rule_full_history(scope, name, vo=vo):
                     yield render_json(**history) + '\n'
 
-            return try_stream(generate(vo=request.environ.get('vo')))
+            return try_stream(generate(vo=request.environ['vo']))
         except ValueError as error:
             return generate_http_error_flask(400, error)
 
@@ -819,7 +819,7 @@ class RuleAnalysis(ErrorHandlingMethodView):
           406:
             description: Not acceptable.
         """
-        analysis = examine_replication_rule(rule_id, issuer=request.environ.get('issuer'), vo=request.environ.get('vo'))
+        analysis = examine_replication_rule(rule_id, issuer=request.environ['issuer'], vo=request.environ['vo'])
         return Response(render_json(**analysis), content_type='application/json')