Skip to content

Issue-456 input configuration should not be altered as it breaks unix…#465

Merged
KostyaSha merged 1 commit intomasterfrom
issue-456-master
Feb 14, 2016
Merged

Issue-456 input configuration should not be altered as it breaks unix…#465
KostyaSha merged 1 commit intomasterfrom
issue-456-master

Conversation

@marcuslinke
Copy link
Copy Markdown
Contributor

… socket support

Fix issue #456 in master branch. Relates to #457

baseResource = client.target(sanitizeUrl(originalUri).toString()).path(dockerClientConfig.getApiVersion().asWebPathPart());
}

private URI sanitizeUrl(URI originalUri) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please provide some javadoc to get idea why it critical.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, don't know what you mean.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well, ok, ignore.

@KostyaSha
Copy link
Copy Markdown
Member

Let's try.

KostyaSha added a commit that referenced this pull request Feb 14, 2016
Issue-456 input configuration should not be altered as it breaks unix…
@KostyaSha KostyaSha merged commit b2de612 into master Feb 14, 2016
@marcuslinke marcuslinke deleted the issue-456-master branch April 8, 2016 18:39
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.

3 participants