Skip to content
New issue

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

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

Already on GitHub? # to your account

Replace manual url concatenation with urllib.parse.urljoin or similar #31

Open
pederhan opened this issue Feb 27, 2023 · 0 comments
Open
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@pederhan
Copy link
Member

Concatenating URLs manually is pretty silly, when there are utility functions for this in the standard library.

https://github.com/pederhan/harborapi/blob/5ce05adc3531f203725222c4e3ffacd03a505582/harborapi/client.py#L3607-L3612

We should replace this with a utility function that all HTTP methods can re-use.

@pederhan pederhan added enhancement New feature or request good first issue Good for newcomers labels Feb 27, 2023
@pederhan pederhan added this to Harbor Feb 27, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant