Skip to content

Commit 8798db2

Browse files
Merge pull request #3633 from renuka-fernando/rl-update
Update Rate Limiter Image to Configure XDS_CLIENT_MAX_MSG_SIZE_IN_BYTES
2 parents 129d843 + 0be2a2a commit 8798db2

File tree

4 files changed

+12
-10
lines changed

4 files changed

+12
-10
lines changed

adapter/internal/oasparser/envoyconf/envoyconf_internal_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -326,8 +326,8 @@ func TestCreateRouteClusterSpecifier(t *testing.T) {
326326

327327
// Test the same scenario with endpointBasePath as empty string and API Resource with root slash
328328
// If endpointBasepath is empty enforce the path to be "/" to avoid setting empty path to upstream.
329-
// https://dev-us-east-azure/dlif/request-info/v1.0 -> https://bachendhost/
330-
// https://dev-us-east-azure/dlif/request-info/v1.0/ -> https://bachendhost/
329+
// https://dev-us-east-azure/dlif/request-info/v1.0 -> https://backendhost/
330+
// https://dev-us-east-azure/dlif/request-info/v1.0/ -> https://backendhost/
331331
routeWithRootSlash := createRoute(generateRouteCreateParamsForUnitTests(title, apiType, vHost, xWso2BasePath, version, "",
332332
"/", resourceWithGet.GetMethodList(), prodClusterName, nil, false))
333333
assert.NotNil(t, routeWithRootSlash, "Route should not be null")
@@ -342,8 +342,8 @@ func TestCreateRouteClusterSpecifier(t *testing.T) {
342342

343343
// if endpointBasepath is not empty, enforce the path to be the endpointBasepath and allow the downnstream to handle trailing slash.
344344
// if downstream sends the trailing slash, send the trailing slash to the upstream otherwise send the path without the trailing slash.
345-
// https://dev-us-east-azure/dlif/request-info/v1.0 -> https://bachendhost/context
346-
// https://dev-us-east-azure/dlif/request-info/v1.0/ -> https://bachendhost/context/
345+
// https://dev-us-east-azure/dlif/request-info/v1.0 -> https://backendhost/context
346+
// https://dev-us-east-azure/dlif/request-info/v1.0/ -> https://backendhost/context/
347347
routeWithRootSlashNonEmptyEp := createRoute(generateRouteCreateParamsForUnitTests(title, apiType, vHost, xWso2BasePath, version, "/foo",
348348
"/", resourceWithGet.GetMethodList(), prodClusterName, nil, false))
349349
assert.NotNil(t, routeWithRootSlashNonEmptyEp, "Route should not be null")

adapter/internal/oasparser/envoyconf/routes_with_clusters.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -952,15 +952,15 @@ func createRoute(params *routeCreateParams) *routev3.Route {
952952
// Hence doing the special logic here.
953953
if endpointBasepath == "" {
954954
// If endpointBasepath is empty enforce the path to be "/" to avoid setting empty path to upstream.
955-
// https://dev-us-east-azure/dlif/request-info/v1.0 -> https://bachendhost/
956-
// https://dev-us-east-azure/dlif/request-info/v1.0/ -> https://bachendhost/
955+
// https://dev-us-east-azure/dlif/request-info/v1.0 -> https://backendhost/
956+
// https://dev-us-east-azure/dlif/request-info/v1.0/ -> https://backendhost/
957957
resourceRegex = "/?"
958958
substitutionString = "/"
959959
} else {
960960
// if endpointBasepath is not empty, enforce the path to be the endpointBasepath and allow the downnstream to handle trailing slash.
961961
// if downstream sends the trailing slash, send the trailing slash to the upstream otherwise send the path without the trailing slash.
962-
// https://dev-us-east-azure/dlif/request-info/v1.0 -> https://bachendhost/context
963-
// https://dev-us-east-azure/dlif/request-info/v1.0/ -> https://bachendhost/context/
962+
// https://dev-us-east-azure/dlif/request-info/v1.0 -> https://backendhost/context
963+
// https://dev-us-east-azure/dlif/request-info/v1.0/ -> https://backendhost/context/
964964
resourceRegex = ""
965965
substitutionString = endpointBasepath
966966
}

enforcer-parent/enforcer/src/main/resources/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
# limitations under the License.
1515
# -----------------------------------------------------------------------
1616

17-
FROM adoptopenjdk/openjdk11:jre-11.0.23_9-alpine
17+
FROM adoptopenjdk/openjdk11:jre-11.0.25_9-alpine
1818
LABEL maintainer="WSO2 Docker Maintainers <wso2.com>"
1919

2020
RUN apk update && apk upgrade --no-cache

rate-limiter/src/main/resources/Dockerfile

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
# limitations under the License.
1515
# -----------------------------------------------------------------------
1616

17-
FROM envoyproxy/ratelimit:19f2079f
17+
FROM envoyproxy/ratelimit:87adad26
1818
LABEL maintainer="WSO2 Docker Maintainers <wso2.com>"
1919

2020
RUN apk update && apk upgrade --no-cache
@@ -45,6 +45,8 @@ ENV CONFIG_GRPC_XDS_CLIENT_TLS_KEY=/home/wso2/security/keystore/rate-limiter.key
4545
ENV CONFIG_GRPC_XDS_CLIENT_TLS_CERT=/home/wso2/security/keystore/rate-limiter.pem
4646
ENV CONFIG_GRPC_XDS_SERVER_TLS_CACERT=/home/wso2/security/truststore/mg.pem
4747

48+
ENV XDS_CLIENT_MAX_MSG_SIZE_IN_BYTES=41943040
49+
4850
ARG MOTD="\n\
4951
Welcome to WSO2 Docker Resources \n\
5052
--------------------------------- \n\

0 commit comments

Comments
 (0)