Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Improve ConnectionManager to handle trailing slash (/) in base_url automatically #707

Open
DanielYang59 wants to merge 2 commits into
marcospereirampj:mastermarcospereirampj/python-keycloak:masterfrom
DanielYang59:handle-server-urlDanielYang59/python-keycloak:handle-server-urlCopy head branch name to clipboard
Open

Improve ConnectionManager to handle trailing slash (/) in base_url automatically #707
DanielYang59 wants to merge 2 commits into
marcospereirampj:mastermarcospereirampj/python-keycloak:masterfrom
DanielYang59:handle-server-urlDanielYang59/python-keycloak:handle-server-urlCopy head branch name to clipboard

Conversation

@DanielYang59
Copy link
Copy Markdown

fix #149

@DanielYang59 DanielYang59 changed the title Improve base_url to handle trailing slash automatically Improve base_url to handle trailing slash (/) automatically Feb 24, 2026
@DanielYang59 DanielYang59 force-pushed the handle-server-url branch 4 times, most recently from c6efc5c to 1ba1daa Compare February 24, 2026 17:05
@DanielYang59 DanielYang59 changed the title Improve base_url to handle trailing slash (/) automatically Improve ConnectionManager to handle trailing slash (/) in base_url automatically Feb 24, 2026
@DanielYang59 DanielYang59 marked this pull request as ready for review February 27, 2026 16:37
@DanielYang59
Copy link
Copy Markdown
Author

DanielYang59 commented Feb 27, 2026

@ryshoooo can you please review this? Here I didn't normalize base_url in the property setter (e.g. add a trailing slash by default), but at the URL building stage, because I'm not sure if there's any case base_url needs to NOT have trailing slash. Thanks

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SUGGESTION: Force the server_url name to end with a '/'

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.