Disallow creating new sessions if client exits context

[cepedus]

Description of Change

Once an AioBaseClient is used as a context manager and exits, forbid any following attempt to create a new session.

If AioBaseClient is never used as a context manager, this restriction does not apply.

Assumptions

Checklist for All Submissions

• I have added change info to CHANGES.rst
• If this is resolving an issue (needed so future developers can determine if change is still necessary and under what conditions) (can be provided via link to issue with these details): Closed client can create non-contextualized connections #1398
  • Detailed description of issue
  • Alternative methods considered (if any)
  • How issue is being resolved
  • How issue can be reproduced
• If this is providing a new feature (can be provided via link to issue with these details):
  • Detailed description of new feature
  • Why needed
  • Alternatives methods considered (if any)

Checklist when updating botocore and/or aiohttp versions

(not applicable)

• I have read and followed CONTRIBUTING.rst
• I have updated test_patches.py where/if appropriate (also check if no changes necessary)
• I have ensured that the awscli/boto3 versions match the updated botocore version
• I have added URL to diff: boto/botocore@[CURRENT_BOTO_VERSION_TAG]...[NEW_BOTO_VERSION_TAG]

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.70%. Comparing base (790af50) to head (42e80bd).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1402      +/-   ##
==========================================
+ Coverage   91.65%   91.70%   +0.04%     
==========================================
  Files          75       76       +1     
  Lines        8067     8078      +11     
==========================================
+ Hits         7394     7408      +14     
+ Misses        673      670       -3     

Flag	Coverage Δ
no-httpx	88.51% <100.00%> (+0.05%)	⬆️
os-ubuntu-24.04	91.70% <100.00%> (+0.04%)	⬆️
os-ubuntu-24.04-arm	89.71% <100.00%> (+0.11%)	⬆️
python-3.10	89.68% <100.00%> (+0.11%)	⬆️
python-3.11	89.68% <100.00%> (+0.11%)	⬆️
python-3.12	89.62% <100.00%> (+0.05%)	⬆️
python-3.13	89.68% <100.00%> (+0.05%)	⬆️
python-3.14	91.67% <100.00%> (+0.11%)	⬆️
python-3.9	89.63% <100.00%> (+0.05%)	⬆️
unittests	91.70% <100.00%> (+0.04%)	⬆️
with-awscrt	91.34% <100.00%> (+0.11%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[thehesiod]

@cepedus you'll need to sign and force push the commits

[jakob-keller]

As I understand it, this PR enforces using AIOHTTPSession as an asynchronous context manager, which was strongly recommended, but optional before. IMO, this might need to be considered a breaking change. Any thoughts? Why is the current situation problematic again?

[cepedus]

@jakob-keller The current behavior is prone to loose sessions that are potentially never closed in the default use-case (RuntimeError potential down the stream if the loop is closed). I agree this could mean a breaking change to those users, which version should I put then?

@thehesiod I signed all commmits, review was dismissed though

[cepedus]

Up, should I continue this MR?

[thehesiod]

@cepedus sorry about delay, been sick. Lets do a major bump to obey semver rules, and merge this puppy

[cepedus]

Bumped major + updated branch :)

[jakob-keller]

@cepedus: Could you please do another update? I am planning to do a 3.0.0 release in the near future and would like to finally include this. Thanks!

[jakob-keller]

Thanks for your patience!