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

Furhtor Refactroing #123

Draft
wants to merge 33 commits into
base: main
Choose a base branch
from

Conversation

Weltraumschaf
Copy link
Member

@Weltraumschaf Weltraumschaf commented Feb 16, 2024

Resolves #121

@Weltraumschaf Weltraumschaf self-assigned this Feb 16, 2024
@Weltraumschaf Weltraumschaf added the enhancement New feature or request label Feb 16, 2024
@Weltraumschaf Weltraumschaf marked this pull request as draft February 16, 2024 15:36
@Weltraumschaf Weltraumschaf force-pushed the 121_furhtor_refactroing branch 2 times, most recently from aaf04ea to 523cf67 Compare March 15, 2024 14:53
Checked exceptions in a public used API is quite cumbersome. Alo
checked exceptions should onlybe used forerrors you can deal with
in the source code. Since the errors in this case origin from outside,
it is better to use runtime exceptions. This does not clutter the API
clients.

Signed-off-by: Sven Strittmatter <[email protected]>
…shable from ProxyConfig

Signed-off-by: Sven Strittmatter <[email protected]>
Signed-off-by: Sven Strittmatter <[email protected]>
Signed-off-by: Sven Strittmatter <[email protected]>
Signed-off-by: Sven Strittmatter <[email protected]>
This has theadvantage that URLs are validated in place and we can
normalize theresulting URL to remove double slashes from path.

Signed-off-by: Sven Strittmatter <[email protected]>
Follow the principle of more details towards the bottom of file.

Signed-off-by: Sven Strittmatter <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Furthor Refactorings DD Client Lib
1 participant