Ocf56 review log level in all services followup

Alex Kakirisrequested to merge
OCF56-review-log-level-in-all-services-followup into staging

This MR is the follow-up to the initial approach validated in the register service (!179 (merged)). The same rules have been applied across all remaining services.

Merge request reports

services/TS29222_CAPIF_API_Invoker_Management_API/api_invoker_management/controllers/individual_on_boarded_api_invoker_document_controller.py

+3 −3
Changes for services/TS29222_CAPIF_API_Invoker_Management_API/api_invoker_management/controllers/individual_on_boarded_api_invoker_document_controller.py: 3 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -56,7 +56,7 @@ def delete_ind_onboarded_api_invoker(onboarding_id): # noqa: E501 


    :rtype: Union[None, Tuple[None, int], Tuple[None, int, Dict[str, str]]

    """

    current_app.logger.info("Removing invoker")

    current_app.logger.debug("Removing invoker")

    res = invoker_operations.remove_apiinvokerenrolmentdetail(onboarding_id)



    return res
@@ -74,7 +74,7 @@ def modify_ind_api_invoke_enrolment(onboarding_id, body): # noqa: E501 


    :rtype: Union[APIInvokerEnrolmentDetails, Tuple[APIInvokerEnrolmentDetails, int], Tuple[APIInvokerEnrolmentDetails, int, Dict[str, str]]

    """

    current_app.logger.info("Updating invoker")

    current_app.logger.debug("Updating invoker")

    if request.is_json:

        body = APIInvokerEnrolmentDetailsPatch.from_dict(request.get_json())  # noqa: E501
@@ -95,7 +95,7 @@ def update_ind_onboarded_api_invoker(onboarding_id, body): # noqa: E501 


    :rtype: Union[APIInvokerEnrolmentDetails, Tuple[APIInvokerEnrolmentDetails, int], Tuple[APIInvokerEnrolmentDetails, int, Dict[str, str]]

    """

    current_app.logger.info("Updating invoker")

    current_app.logger.debug("Updating invoker")

    if request.is_json:

        body = APIInvokerEnrolmentDetails.from_dict(request.get_json())  # noqa: E501

services/TS29222_CAPIF_API_Invoker_Management_API/api_invoker_management/controllers/on_boarded_api_invokers_collection_controller.py

+1 −1
Changes for services/TS29222_CAPIF_API_Invoker_Management_API/api_invoker_management/controllers/on_boarded_api_invokers_collection_controller.py: 1 added line, 1 removed line.
Original line number Diff line number Diff line
@@ -24,7 +24,7 @@ def create_onboarded_api_invoker(body): # noqa: E501 
    identity = get_jwt_identity()

    username, uuid = identity.split()



    current_app.logger.info("Creating Invoker")

    current_app.logger.debug("Creating Invoker")

    if request.is_json:

        body = APIInvokerEnrolmentDetails.from_dict(request.get_json())  # noqa: E501

services/TS29222_CAPIF_API_Invoker_Management_API/api_invoker_management/core/apiinvokerenrolmentdetails.py

+3 −3
Changes for services/TS29222_CAPIF_API_Invoker_Management_API/api_invoker_management/core/apiinvokerenrolmentdetails.py: 3 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -51,7 +51,7 @@ class InvokerManagementOperations(Resource): 
        old_values = mycol.find_one(my_query)



        if old_values is None:

            current_app.logger.error("Not found api invoker id")

            current_app.logger.warning("Not found api invoker id")

            return not_found_error(detail="Please provide an existing Network App ID", cause="Not exist Network App ID")



        return old_values
@@ -93,12 +93,12 @@ class InvokerManagementOperations(Resource): 
                             apiinvokerenrolmentdetail.onboarding_information.api_invoker_public_key})



        if res is not None:

            current_app.logger.error(

            current_app.logger.warning(

                "Generating forbbiden error, invoker registered")

            return forbidden_error(detail="Invoker already registered", cause="Identical invoker public key")



        if rfc3987.match(apiinvokerenrolmentdetail.notification_destination, rule="URI") is None:

            current_app.logger.error("Bad url format")

            current_app.logger.warning("Bad url format")

            return bad_request_error(detail="Bad Param", cause="Detected Bad formar of param", invalid_params=[{"param": "notificationDestination", "reason": "Not valid URL format"}])



        if not apiinvokerenrolmentdetail.supported_features:

services/TS29222_CAPIF_API_Invoker_Management_API/api_invoker_management/core/consumer_messager.py

+1 −1
Changes for services/TS29222_CAPIF_API_Invoker_Management_API/api_invoker_management/core/consumer_messager.py: 1 added line, 1 removed line.
Original line number Diff line number Diff line
@@ -18,7 +18,7 @@ class Subscriber(): 
    def listen(self):

        for raw_message in self.p.listen():

            if raw_message["type"] == "message" and raw_message["channel"].decode('utf-8') == "internal-messages":

                current_app.logger.info("New internal event received")

                current_app.logger.debug("New internal event received")

                internal_redis_event = json.loads(

                    raw_message["data"].decode('utf-8'))

                if internal_redis_event.get('event') == "SECURITY-CONTEXT-CREATED":

services/TS29222_CAPIF_API_Provider_Management_API/api_provider_management/controllers/default_controller.py

+3 −3
Changes for services/TS29222_CAPIF_API_Provider_Management_API/api_provider_management/controllers/default_controller.py: 3 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -60,7 +60,7 @@ def registrations_post(body): # noqa: E501 
    identity = get_jwt_identity()

    username, uuid = identity.split()



    current_app.logger.info("Registering Provider Domain")

    current_app.logger.debug("Registering Provider Domain")



    if request.is_json:

        body = APIProviderEnrolmentDetails.from_dict(request.get_json())  # noqa: E501
@@ -80,7 +80,7 @@ def registrations_registration_id_delete(registration_id): # noqa: E501 


    :rtype: Union[None, Tuple[None, int], Tuple[None, int, Dict[str, str]]

    """

    current_app.logger.info("Removing Provider Domain")

    current_app.logger.debug("Removing Provider Domain")

    res = provider_management_ops.delete_api_provider_enrolment_details(registration_id)



    return res
@@ -98,7 +98,7 @@ def registrations_registration_id_put(registration_id, body): # noqa: E501 


    :rtype: Union[APIProviderEnrolmentDetails, Tuple[APIProviderEnrolmentDetails, int], Tuple[APIProviderEnrolmentDetails, int, Dict[str, str]]

    """

    current_app.logger.info("Updating Provider Domain")

    current_app.logger.debug("Updating Provider Domain")



    if request.is_json:

        body = APIProviderEnrolmentDetails.from_dict(request.get_json())  # noqa: E501