Skip to content
This repository was archived by the owner on Dec 17, 2025. It is now read-only.

Refactor to use yarl package for safe URL manipulation#74

Open
rapsealk wants to merge 3 commits intoray-project:mainfrom
rapsealk:feature/use-yarl-to-manipulate-url
Open

Refactor to use yarl package for safe URL manipulation#74
rapsealk wants to merge 3 commits intoray-project:mainfrom
rapsealk:feature/use-yarl-to-manipulate-url

Conversation

@rapsealk
Copy link

@rapsealk rapsealk commented Oct 15, 2024

Hello llmperf!

This pull request suggests to use yarl, which is already included, to manipulate URL safely.

The change in this pull request, yarl.URL(address).with_path("/chat/completions"), will always result in a valid URL, whether the address ends with a trailing slash(/) or not.

Thanks in advance for your kind review!

Ref

@rapsealk
Copy link
Author

rapsealk commented Oct 27, 2024

Hi @waleedkadous! Would you mind if I ask for a review? Thanks in advance.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant