Skip to content

Commit e19cf64

Browse files
retareta
authored
[FEATURE] Built-in secure transports support (opensearch-project#12435) (opensearch-project#12789) (opensearch-project#12794)
* [FEATURE] Built-in secure transports support Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Added more tests, addressing code review comments Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Address code review comments Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Address code review comments Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Address code review comments Signed-off-by: Andriy Redko <andriy.redko@aiven.io> --------- Signed-off-by: Andriy Redko <andriy.redko@aiven.io> (cherry picked from commit 773a939) (cherry picked from commit 3cc154a)
1 parent 0560d99 commit e19cf64

23 files changed

+2237
-7
lines changed

‎CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
2727
- [Tiered caching] Add serializer integration to allow ehcache disk cache to use non-primitive values ([#12709](https://github.com/opensearch-project/OpenSearch/pull/12709))
2828
- [Admission Control] Integrated IO Based AdmissionController to AdmissionControl Framework ([#12583](https://github.com/opensearch-project/OpenSearch/pull/12583))
2929
- Add Remote Store Migration Experimental flag and allow mixed mode clusters under same ([#11986](https://github.com/opensearch-project/OpenSearch/pull/11986))
30+
- Built-in secure transports support ([#12435](https://github.com/opensearch-project/OpenSearch/pull/12435))
3031

3132
### Dependencies
3233
- Bump `com.squareup.okio:okio` from 3.7.0 to 3.8.0 ([#12290](https://github.com/opensearch-project/OpenSearch/pull/12290))

‎modules/transport-netty4/src/main/java/org/opensearch/http/netty4/ssl/SecureNetty4HttpServerTransport.java

+126
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,126 @@
1+
/*
2+
* Copyright 2015-2017 floragunn GmbH
3+
*
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
*
8+
* http://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
15+
*
16+
*/
17+
18+
/*
19+
* SPDX-License-Identifier: Apache-2.0
20+
*
21+
* The OpenSearch Contributors require contributions made to
22+
* this file be licensed under the Apache-2.0 license or a
23+
* compatible open source license.
24+
*
25+
* Modifications Copyright OpenSearch Contributors. See
26+
* GitHub history for details.
27+
*/
28+
29+
package org.opensearch.http.netty4.ssl;
30+
31+
import org.apache.logging.log4j.LogManager;
32+
import org.apache.logging.log4j.Logger;
33+
import org.opensearch.common.network.NetworkService;
34+
import org.opensearch.common.settings.ClusterSettings;
35+
import org.opensearch.common.settings.Settings;
36+
import org.opensearch.common.util.BigArrays;
37+
import org.opensearch.core.xcontent.NamedXContentRegistry;
38+
import org.opensearch.http.HttpChannel;
39+
import org.opensearch.http.HttpHandlingSettings;
40+
import org.opensearch.http.netty4.Netty4HttpServerTransport;
41+
import org.opensearch.plugins.SecureTransportSettingsProvider;
42+
import org.opensearch.telemetry.tracing.Tracer;
43+
import org.opensearch.threadpool.ThreadPool;
44+
import org.opensearch.transport.SharedGroupFactory;
45+
import org.opensearch.transport.netty4.ssl.SslUtils;
46+
47+
import javax.net.ssl.SSLEngine;
48+
49+
import io.netty.channel.Channel;
50+
import io.netty.channel.ChannelHandler;
51+
import io.netty.handler.codec.DecoderException;
52+
import io.netty.handler.ssl.SslHandler;
53+
54+
/**
55+
* @see <a href="https://github.com/opensearch-project/security/blob/d526c9f6c2a438c14db8b413148204510b9fe2e2/src/main/java/org/opensearch/security/ssl/http/netty/SecuritySSLNettyHttpServerTransport.java">SecuritySSLNettyHttpServerTransport</a>
56+
*/
57+
public class SecureNetty4HttpServerTransport extends Netty4HttpServerTransport {
58+
private static final Logger logger = LogManager.getLogger(SecureNetty4HttpServerTransport.class);
59+
private final SecureTransportSettingsProvider secureTransportSettingsProvider;
60+
private final SecureTransportSettingsProvider.ServerExceptionHandler exceptionHandler;
61+
62+
public SecureNetty4HttpServerTransport(
63+
final Settings settings,
64+
final NetworkService networkService,
65+
final BigArrays bigArrays,
66+
final ThreadPool threadPool,
67+
final NamedXContentRegistry namedXContentRegistry,
68+
final Dispatcher dispatcher,
69+
final ClusterSettings clusterSettings,
70+
final SharedGroupFactory sharedGroupFactory,
71+
final SecureTransportSettingsProvider secureTransportSettingsProvider,
72+
final Tracer tracer
73+
) {
74+
super(
75+
settings,
76+
networkService,
77+
bigArrays,
78+
threadPool,
79+
namedXContentRegistry,
80+
dispatcher,
81+
clusterSettings,
82+
sharedGroupFactory,
83+
tracer
84+
);
85+
this.secureTransportSettingsProvider = secureTransportSettingsProvider;
86+
this.exceptionHandler = secureTransportSettingsProvider.buildHttpServerExceptionHandler(settings, this)
87+
.orElse(SecureTransportSettingsProvider.ServerExceptionHandler.NOOP);
88+
}
89+
90+
@Override
91+
public ChannelHandler configureServerChannelHandler() {
92+
return new SslHttpChannelHandler(this, handlingSettings);
93+
}
94+
95+
@Override
96+
public void onException(HttpChannel channel, Exception cause0) {
97+
Throwable cause = cause0;
98+
99+
if (cause0 instanceof DecoderException && cause0 != null) {
100+
cause = cause0.getCause();
101+
}
102+
103+
exceptionHandler.onError(cause);
104+
logger.error("Exception during establishing a SSL connection: " + cause, cause);
105+
super.onException(channel, cause0);
106+
}
107+
108+
protected class SslHttpChannelHandler extends Netty4HttpServerTransport.HttpChannelHandler {
109+
protected SslHttpChannelHandler(final Netty4HttpServerTransport transport, final HttpHandlingSettings handlingSettings) {
110+
super(transport, handlingSettings);
111+
}
112+
113+
@Override
114+
protected void initChannel(Channel ch) throws Exception {
115+
super.initChannel(ch);
116+
117+
final SSLEngine sslEngine = secureTransportSettingsProvider.buildSecureHttpServerEngine(
118+
settings,
119+
SecureNetty4HttpServerTransport.this
120+
).orElseGet(SslUtils::createDefaultServerSSLEngine);
121+
122+
final SslHandler sslHandler = new SslHandler(sslEngine);
123+
ch.pipeline().addFirst("ssl_http", sslHandler);
124+
}
125+
}
126+
}

‎modules/transport-netty4/src/main/java/org/opensearch/transport/Netty4Plugin.java

+64
Original file line numberDiff line numberDiff line change
@@ -46,11 +46,14 @@
4646
import org.opensearch.core.xcontent.NamedXContentRegistry;
4747
import org.opensearch.http.HttpServerTransport;
4848
import org.opensearch.http.netty4.Netty4HttpServerTransport;
49+
import org.opensearch.http.netty4.ssl.SecureNetty4HttpServerTransport;
4950
import org.opensearch.plugins.NetworkPlugin;
5051
import org.opensearch.plugins.Plugin;
52+
import org.opensearch.plugins.SecureTransportSettingsProvider;
5153
import org.opensearch.telemetry.tracing.Tracer;
5254
import org.opensearch.threadpool.ThreadPool;
5355
import org.opensearch.transport.netty4.Netty4Transport;
56+
import org.opensearch.transport.netty4.ssl.SecureNetty4Transport;
5457

5558
import java.util.Arrays;
5659
import java.util.Collections;
@@ -61,7 +64,9 @@
6164
public class Netty4Plugin extends Plugin implements NetworkPlugin {
6265

6366
public static final String NETTY_TRANSPORT_NAME = "netty4";
67+
public static final String NETTY_SECURE_TRANSPORT_NAME = "netty4-secure";
6468
public static final String NETTY_HTTP_TRANSPORT_NAME = "netty4";
69+
public static final String NETTY_SECURE_HTTP_TRANSPORT_NAME = "netty4-secure";
6570

6671
private final SetOnce<SharedGroupFactory> groupFactory = new SetOnce<>();
6772

@@ -144,6 +149,65 @@ public Map<String, Supplier<HttpServerTransport>> getHttpTransports(
144149
);
145150
}
146151

152+
@Override
153+
public Map<String, Supplier<HttpServerTransport>> getSecureHttpTransports(
154+
Settings settings,
155+
ThreadPool threadPool,
156+
BigArrays bigArrays,
157+
PageCacheRecycler pageCacheRecycler,
158+
CircuitBreakerService circuitBreakerService,
159+
NamedXContentRegistry xContentRegistry,
160+
NetworkService networkService,
161+
HttpServerTransport.Dispatcher dispatcher,
162+
ClusterSettings clusterSettings,
163+
SecureTransportSettingsProvider secureTransportSettingsProvider,
164+
Tracer tracer
165+
) {
166+
return Collections.singletonMap(
167+
NETTY_SECURE_HTTP_TRANSPORT_NAME,
168+
() -> new SecureNetty4HttpServerTransport(
169+
settings,
170+
networkService,
171+
bigArrays,
172+
threadPool,
173+
xContentRegistry,
174+
dispatcher,
175+
clusterSettings,
176+
getSharedGroupFactory(settings),
177+
secureTransportSettingsProvider,
178+
tracer
179+
)
180+
);
181+
}
182+
183+
@Override
184+
public Map<String, Supplier<Transport>> getSecureTransports(
185+
Settings settings,
186+
ThreadPool threadPool,
187+
PageCacheRecycler pageCacheRecycler,
188+
CircuitBreakerService circuitBreakerService,
189+
NamedWriteableRegistry namedWriteableRegistry,
190+
NetworkService networkService,
191+
SecureTransportSettingsProvider secureTransportSettingsProvider,
192+
Tracer tracer
193+
) {
194+
return Collections.singletonMap(
195+
NETTY_SECURE_TRANSPORT_NAME,
196+
() -> new SecureNetty4Transport(
197+
settings,
198+
Version.CURRENT,
199+
threadPool,
200+
networkService,
201+
pageCacheRecycler,
202+
namedWriteableRegistry,
203+
circuitBreakerService,
204+
getSharedGroupFactory(settings),
205+
secureTransportSettingsProvider,
206+
tracer
207+
)
208+
);
209+
}
210+
147211
SharedGroupFactory getSharedGroupFactory(Settings settings) {
148212
SharedGroupFactory groupFactory = this.groupFactory.get();
149213
if (groupFactory != null) {

‎modules/transport-netty4/src/main/java/org/opensearch/transport/netty4/ssl/DualModeSslHandler.java

+106
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,106 @@
1+
/*
2+
* SPDX-License-Identifier: Apache-2.0
3+
*
4+
* The OpenSearch Contributors require contributions made to
5+
* this file be licensed under the Apache-2.0 license or a
6+
* compatible open source license.
7+
*
8+
* Modifications Copyright OpenSearch Contributors. See
9+
* GitHub history for details.
10+
*/
11+
package org.opensearch.transport.netty4.ssl;
12+
13+
import org.apache.logging.log4j.LogManager;
14+
import org.apache.logging.log4j.Logger;
15+
import org.opensearch.common.settings.Settings;
16+
import org.opensearch.plugins.SecureTransportSettingsProvider;
17+
import org.opensearch.transport.TcpTransport;
18+
19+
import javax.net.ssl.SSLEngine;
20+
import javax.net.ssl.SSLException;
21+
22+
import java.nio.charset.StandardCharsets;
23+
import java.security.NoSuchAlgorithmException;
24+
import java.util.List;
25+
26+
import io.netty.buffer.ByteBuf;
27+
import io.netty.buffer.Unpooled;
28+
import io.netty.channel.ChannelFutureListener;
29+
import io.netty.channel.ChannelHandlerContext;
30+
import io.netty.channel.ChannelPipeline;
31+
import io.netty.handler.codec.ByteToMessageDecoder;
32+
import io.netty.handler.ssl.SslHandler;
33+
34+
/**
35+
* Modifies the current pipeline dynamically to enable TLS
36+
*
37+
* @see <a href="https://github.com/opensearch-project/security/blob/d526c9f6c2a438c14db8b413148204510b9fe2e2/src/main/java/org/opensearch/security/ssl/transport/DualModeSSLHandler.java">DualModeSSLHandler</a>
38+
*/
39+
public class DualModeSslHandler extends ByteToMessageDecoder {
40+
41+
private static final Logger logger = LogManager.getLogger(DualModeSslHandler.class);
42+
private final Settings settings;
43+
private final SecureTransportSettingsProvider secureTransportSettingsProvider;
44+
private final TcpTransport transport;
45+
private final SslHandler providedSSLHandler;
46+
47+
public DualModeSslHandler(
48+
final Settings settings,
49+
final SecureTransportSettingsProvider secureTransportSettingsProvider,
50+
final TcpTransport transport
51+
) {
52+
this(settings, secureTransportSettingsProvider, transport, null);
53+
}
54+
55+
protected DualModeSslHandler(
56+
final Settings settings,
57+
final SecureTransportSettingsProvider secureTransportSettingsProvider,
58+
final TcpTransport transport,
59+
SslHandler providedSSLHandler
60+
) {
61+
this.settings = settings;
62+
this.secureTransportSettingsProvider = secureTransportSettingsProvider;
63+
this.transport = transport;
64+
this.providedSSLHandler = providedSSLHandler;
65+
}
66+
67+
@Override
68+
protected void decode(ChannelHandlerContext ctx, ByteBuf in, List<Object> out) throws Exception {
69+
// Will use the first six bytes to detect a protocol.
70+
if (in.readableBytes() < 6) {
71+
return;
72+
}
73+
int offset = in.readerIndex();
74+
if (in.getCharSequence(offset, 6, StandardCharsets.UTF_8).equals(SecureConnectionTestUtil.DUAL_MODE_CLIENT_HELLO_MSG)) {
75+
logger.debug("Received DualSSL Client Hello message");
76+
ByteBuf responseBuffer = Unpooled.buffer(6);
77+
responseBuffer.writeCharSequence(SecureConnectionTestUtil.DUAL_MODE_SERVER_HELLO_MSG, StandardCharsets.UTF_8);
78+
ctx.writeAndFlush(responseBuffer).addListener(ChannelFutureListener.CLOSE);
79+
return;
80+
}
81+
82+
if (SslUtils.isTLS(in)) {
83+
logger.debug("Identified request as SSL request");
84+
enableSsl(ctx);
85+
} else {
86+
logger.debug("Identified request as non SSL request, running in HTTP mode as dual mode is enabled");
87+
ctx.pipeline().remove(this);
88+
}
89+
}
90+
91+
private void enableSsl(ChannelHandlerContext ctx) throws SSLException, NoSuchAlgorithmException {
92+
final SSLEngine sslEngine = secureTransportSettingsProvider.buildSecureServerTransportEngine(settings, transport)
93+
.orElseGet(SslUtils::createDefaultServerSSLEngine);
94+
95+
SslHandler sslHandler;
96+
if (providedSSLHandler != null) {
97+
sslHandler = providedSSLHandler;
98+
} else {
99+
sslHandler = new SslHandler(sslEngine);
100+
}
101+
ChannelPipeline p = ctx.pipeline();
102+
p.addAfter("port_unification_handler", "ssl_server", sslHandler);
103+
p.remove(this);
104+
logger.debug("Removed port unification handler and added SSL handler as incoming request is SSL");
105+
}
106+
}

0 commit comments

Comments
 (0)