Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

path_params and url format not work #6753

Closed
2 of 14 tasks
0xbeekeeper opened this issue Jun 26, 2023 · 1 comment
Closed
2 of 14 tasks

path_params and url format not work #6753

0xbeekeeper opened this issue Jun 26, 2023 · 1 comment
Labels
01 bug Confirmed bug

Comments

@0xbeekeeper
Copy link

System Info

Python 3.9.6

Requirement

aiohttp==3.8.4
aiosignal==1.3.1
async-timeout==4.0.2
attrs==23.1.0
certifi==2023.5.7
charset-normalizer==3.1.0
dataclasses-json==0.5.8
docopt==0.6.2
frozenlist==1.3.3
idna==3.4
langchain==0.0.215
langchainplus-sdk==0.0.17
marshmallow==3.19.0
marshmallow-enum==1.5.1
multidict==6.0.4
mypy-extensions==1.0.0
numexpr==2.8.4
numpy==1.25.0
openai==0.27.8
openapi-schema-pydantic==1.2.4
packaging==23.1
pipreqs==0.4.13
pydantic==1.10.9
PyYAML==6.0
requests==2.31.0
SQLAlchemy==2.0.17
tenacity==8.2.2
tqdm==4.65.0
typing-inspect==0.9.0
typing_extensions==4.6.3
urllib3==2.0.3
yarg==0.1.9
yarl==1.9.2

Who can help?

No response

Information

  • The official example notebooks/scripts
  • My own modified scripts

Related Components

  • LLMs/Chat Models
  • Embedding Models
  • Prompts / Prompt Templates / Prompt Selectors
  • Output Parsers
  • Document Loaders
  • Vector Stores / Retrievers
  • Memory
  • Agents / Agent Executors
  • Tools / Toolkits
  • Chains
  • Callbacks/Tracing
  • Async

Reproduction

Endpoint:

URL: https://api.gopluslabs.io//api/v1/token_security/{chain_id}
arguments:
image

In openapi.py: 160
image
After the _format_url, url doesn't change, the result of printer is also https://api.gopluslabs.io//api/v1/token_security/{chain_id}

Expected behavior

Expect URL and path parameters to be properly combined after formatting like:
https://api.gopluslabs.io//api/v1/token_security/1

@dosubot dosubot bot added the 01 bug Confirmed bug label Jun 26, 2023
hwchase17 pushed a commit that referenced this issue Jun 27, 2023
- Description: format the url and path_params correctly, 
  - Issue: #6753,
  - Dependencies: None,
  - Tag maintainer: @vowelparrot,
  - Twitter handle: @0xbluesecurity
vowelparrot pushed a commit that referenced this issue Jul 4, 2023
- Description: format the url and path_params correctly, 
  - Issue: #6753,
  - Dependencies: None,
  - Tag maintainer: @vowelparrot,
  - Twitter handle: @0xbluesecurity
@dosubot
Copy link

dosubot bot commented Sep 25, 2023

Hi, @0xbluesecurity! I'm Dosu, and I'm here to help the LangChain team manage their backlog. I wanted to let you know that we are marking this issue as stale.

From what I understand, you opened this issue regarding the incorrect behavior of path_params and URL formatting in the code. The expected outcome is for the URL and path parameters to be combined accurately after formatting. However, there haven't been any comments or updates on the issue since you opened it.

Could you please let us know if this issue is still relevant to the latest version of the LangChain repository? If it is, please comment on the issue to let the LangChain team know. Otherwise, feel free to close the issue yourself, or the issue will be automatically closed in 7 days.

Thank you for your understanding and contribution to the LangChain project!

@dosubot dosubot bot added the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Sep 25, 2023
@dosubot dosubot bot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 2, 2023
@dosubot dosubot bot removed the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
01 bug Confirmed bug
Projects
None yet
Development

No branches or pull requests

1 participant