fix: do not instantiate logger inside a method

[leninmehedy]

Description:
This PR removes the problematic instantiation of a logger inside a method that is invoked repeatedly.

Related issue(s):

Fixes #2194

Notes for reviewer:

I couldn't run the integration tests as there were no instruction in the README. I saw errors when I ran task test locally: Error: Failed to construct client for IntegrationTestEnv

However, please let me know if anything else I should test/modify.

Checklist

• Documented (Code comments, README, etc.)
• Tested (unit, integration, etc.)

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.47%. Comparing base (e4a80a1) to head (d875cdd).

❗ Current head d875cdd differs from pull request most recent head 73ec899. Consider uploading reports for the commit 73ec899 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2195      +/-   ##
===========================================
- Coverage    84.48%   84.47%   -0.02%     
===========================================
  Files          305      305              
  Lines        76259    76231      -28     
===========================================
- Hits         64430    64394      -36     
- Misses       11829    11837       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[svetoslav-nikol0v]

Hey leninmehedy,
This logger was only added to track the behavior after the fix we added. We agree to remove it at all. Will add a commit for this.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

[agadzhalov]

LGTM