Fix issue where requests wrapper passes auth kwarg twice (#11010)

<!-- Thank you for contributing to LangChain!

Replace this entire comment with:
  - **Description:** a description of the change, 
  - **Issue:** the issue # it fixes (if applicable),
  - **Dependencies:** any dependencies required for this change,
- **Tag maintainer:** for a quicker response, tag the relevant
maintainer (see below),
- **Twitter handle:** we announce bigger features on Twitter. If your PR
gets announced, and you'd like a mention, we'll gladly shout you out!

Please make sure your PR is passing linting and testing before
submitting. Run `make format`, `make lint` and `make test` to check this
locally.

See contribution guidelines for more information on how to write/run
tests, lint, etc:

https://github.com/hwchase17/langchain/blob/master/.github/CONTRIBUTING.md

If you're adding a new integration, please include:
1. a test for the integration, preferably unit tests that do not rely on
network access,
2. an example notebook showing its use. It lives in `docs/extras`
directory.

If no one reviews your PR within a few days, please @-mention one of
@baskaryan, @eyurtsev, @hwchase17.
 -->

Closes #8842
This commit is contained in:
Nuno Campos 2023-09-25 15:45:04 +01:00 committed by GitHub
parent 88a02076af
commit 956ee981c0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -73,7 +73,7 @@ class Requests(BaseModel):
self, url: str, **kwargs: Any self, url: str, **kwargs: Any
) -> AsyncGenerator[aiohttp.ClientResponse, None]: ) -> AsyncGenerator[aiohttp.ClientResponse, None]:
"""GET the URL and return the text asynchronously.""" """GET the URL and return the text asynchronously."""
async with self._arequest("GET", url, auth=self.auth, **kwargs) as response: async with self._arequest("GET", url, **kwargs) as response:
yield response yield response
@asynccontextmanager @asynccontextmanager
@ -81,9 +81,7 @@ class Requests(BaseModel):
self, url: str, data: Dict[str, Any], **kwargs: Any self, url: str, data: Dict[str, Any], **kwargs: Any
) -> AsyncGenerator[aiohttp.ClientResponse, None]: ) -> AsyncGenerator[aiohttp.ClientResponse, None]:
"""POST to the URL and return the text asynchronously.""" """POST to the URL and return the text asynchronously."""
async with self._arequest( async with self._arequest("POST", url, json=data, **kwargs) as response:
"POST", url, json=data, auth=self.auth, **kwargs
) as response:
yield response yield response
@asynccontextmanager @asynccontextmanager
@ -91,9 +89,7 @@ class Requests(BaseModel):
self, url: str, data: Dict[str, Any], **kwargs: Any self, url: str, data: Dict[str, Any], **kwargs: Any
) -> AsyncGenerator[aiohttp.ClientResponse, None]: ) -> AsyncGenerator[aiohttp.ClientResponse, None]:
"""PATCH the URL and return the text asynchronously.""" """PATCH the URL and return the text asynchronously."""
async with self._arequest( async with self._arequest("PATCH", url, json=data, **kwargs) as response:
"PATCH", url, json=data, auth=self.auth, **kwargs
) as response:
yield response yield response
@asynccontextmanager @asynccontextmanager
@ -101,9 +97,7 @@ class Requests(BaseModel):
self, url: str, data: Dict[str, Any], **kwargs: Any self, url: str, data: Dict[str, Any], **kwargs: Any
) -> AsyncGenerator[aiohttp.ClientResponse, None]: ) -> AsyncGenerator[aiohttp.ClientResponse, None]:
"""PUT the URL and return the text asynchronously.""" """PUT the URL and return the text asynchronously."""
async with self._arequest( async with self._arequest("PUT", url, json=data, **kwargs) as response:
"PUT", url, json=data, auth=self.auth, **kwargs
) as response:
yield response yield response
@asynccontextmanager @asynccontextmanager
@ -111,7 +105,7 @@ class Requests(BaseModel):
self, url: str, **kwargs: Any self, url: str, **kwargs: Any
) -> AsyncGenerator[aiohttp.ClientResponse, None]: ) -> AsyncGenerator[aiohttp.ClientResponse, None]:
"""DELETE the URL and return the text asynchronously.""" """DELETE the URL and return the text asynchronously."""
async with self._arequest("DELETE", url, auth=self.auth, **kwargs) as response: async with self._arequest("DELETE", url, **kwargs) as response:
yield response yield response