mirror of
https://github.com/Significant-Gravitas/AutoGPT.git
synced 2026-04-08 03:00:28 -04:00
feat(backend): Support aiohttp.BasicAuth in make_request (#10283)
Fixes https://github.com/Significant-Gravitas/AutoGPT/issues/10284 ### Changes 🏗️ - Allows passing an `aiohttp.BasicAuth` object directly to the `auth` parameter of the `make_request` function. - Converts tuple-based auth credentials to `aiohttp.BasicAuth` objects before making the request. Fixes [AUTOGPT-SERVER-4AX](https://sentry.io/organizations/significant-gravitas/issues/6709824432/). The issue was that: aiohttp's ClientSession.request received a plain tuple for `auth` instead of an `aiohttp.BasicAuth` object, causing OAuth2 token exchange failure. This fix was generated by Seer in Sentry, triggered by Bently. 👁️ Run ID: 185767 Not quite right? [Click here to continue debugging with Seer.](https://sentry.io/organizations/significant-gravitas/issues/6709824432/?seerDrawer=true) ### Checklist 📋 #### For code changes: - [x] I have clearly listed my changes in the PR description - [x] I have made a test plan - [x] I have tested my changes according to the test plan #### For configuration changes: - [x] `.env.example` is updated or already compatible with my changes - [x] `docker-compose.yml` is updated or already compatible with my changes - [x] I have included a list of my configuration changes in the PR description (under **Changes**) <details> <summary>Examples of configuration changes</summary> - Changing ports - Adding new services that need to communicate with each other - Secrets or environment variable changes - New or infrastructure changes such as databases </details> Co-authored-by: seer-by-sentry[bot] <157164994+seer-by-sentry[bot]@users.noreply.github.com>
This commit is contained in:
committed by
GitHub
parent
22d58367ec
commit
47f503f223
@@ -353,6 +353,10 @@ class Requests:
|
||||
max_redirects: int = 10,
|
||||
**kwargs,
|
||||
) -> Response:
|
||||
# Convert auth tuple to aiohttp.BasicAuth if necessary
|
||||
if "auth" in kwargs and isinstance(kwargs["auth"], tuple):
|
||||
kwargs["auth"] = aiohttp.BasicAuth(*kwargs["auth"])
|
||||
|
||||
if files is not None:
|
||||
if json is not None:
|
||||
raise ValueError(
|
||||
|
||||
Reference in New Issue
Block a user