Skip to content

Conversation

@marcuslinke
Copy link
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
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
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
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
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