-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Add support of SOCKS proxies for S3 repository #2160
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,123 @@ | ||
/* | ||
* SPDX-License-Identifier: Apache-2.0 | ||
* | ||
* The OpenSearch Contributors require contributions made to | ||
* this file be licensed under the Apache-2.0 license or a | ||
* compatible open source license. | ||
*/ | ||
|
||
package org.opensearch.repositories.s3; | ||
|
||
import com.amazonaws.Protocol; | ||
import org.opensearch.common.Strings; | ||
import org.opensearch.common.settings.SettingsException; | ||
|
||
import java.net.InetAddress; | ||
import java.net.InetSocketAddress; | ||
import java.net.UnknownHostException; | ||
import java.util.Objects; | ||
|
||
public class ProxySettings { | ||
public static final ProxySettings NO_PROXY_SETTINGS = new ProxySettings(ProxyType.DIRECT, null, -1, null, null); | ||
|
||
public static enum ProxyType { | ||
HTTP(Protocol.HTTP.name()), | ||
HTTPS(Protocol.HTTPS.name()), | ||
SOCKS("SOCKS"), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we actually need to introduce There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is possible to support SOCKS. Socks woks on the Socket level. So extending SSLSocketFactory is ok IMHO There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nevermind, I got that, conversion to protocol is not supported for SOCKS |
||
DIRECT("DIRECT"); | ||
|
||
private final String name; | ||
|
||
private ProxyType(String name) { | ||
this.name = name; | ||
} | ||
|
||
public Protocol toProtocol() { | ||
if (this == DIRECT) { | ||
// We check it in settings, | ||
// the probability that it could be thrown is small, but how knows | ||
throw new SettingsException("Couldn't convert to S3 protocol"); | ||
} else if (this == SOCKS) { | ||
throw new SettingsException("Couldn't convert to S3 protocol. SOCKS is not supported"); | ||
} | ||
return Protocol.valueOf(name()); | ||
} | ||
|
||
} | ||
|
||
private final ProxyType type; | ||
|
||
private final String host; | ||
|
||
private final String username; | ||
|
||
private final String password; | ||
|
||
private final int port; | ||
|
||
public String getHost() { | ||
return host; | ||
} | ||
|
||
public ProxySettings(final ProxyType type, final String host, final int port, final String username, final String password) { | ||
this.type = type; | ||
this.host = host; | ||
this.port = port; | ||
this.username = username; | ||
this.password = password; | ||
} | ||
|
||
public ProxyType getType() { | ||
return this.type; | ||
} | ||
|
||
public String getHostName() { | ||
return host; | ||
} | ||
|
||
public int getPort() { | ||
return port; | ||
} | ||
|
||
public InetSocketAddress getAddress() { | ||
try { | ||
return new InetSocketAddress(InetAddress.getByName(host), port); | ||
} catch (UnknownHostException e) { | ||
// this error won't be thrown since validation of the host name is in the S3ClientSettings | ||
throw new RuntimeException(e); | ||
} | ||
} | ||
|
||
public String getUsername() { | ||
return this.username; | ||
} | ||
|
||
public String getPassword() { | ||
return this.password; | ||
} | ||
|
||
public boolean isAuthenticated() { | ||
return Strings.isNullOrEmpty(username) == false && Strings.isNullOrEmpty(password) == false; | ||
} | ||
|
||
public ProxySettings recreateWithNewHostAndPort(final String host, final int port) { | ||
return new ProxySettings(type, host, port, username, password); | ||
} | ||
|
||
@Override | ||
public boolean equals(Object o) { | ||
if (this == o) return true; | ||
if (o == null || getClass() != o.getClass()) return false; | ||
final ProxySettings that = (ProxySettings) o; | ||
return port == that.port | ||
&& type == that.type | ||
&& Objects.equals(host, that.host) | ||
&& Objects.equals(username, that.username) | ||
&& Objects.equals(password, that.password); | ||
} | ||
|
||
@Override | ||
public int hashCode() { | ||
return Objects.hash(type, host, username, password, port); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the same solution as for others repositories. Can be extracted to a common part
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tracked in #2155