Skip to content

Commit

Permalink
fix linter
Browse files Browse the repository at this point in the history
  • Loading branch information
Ferdudas97 committed Jan 15, 2024
1 parent 03c0e70 commit 2968fb2
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,8 @@ fun getPathNormalization(proto: Value?, snapshotProperties: SnapshotProperties):
return PathNormalizationConfig(
normalizationEnabled = proto.field("enabled")?.boolValue ?: defaultNormalizationConfig.normalizationEnabled,
mergeSlashes = proto.field("merge_slashes")?.boolValue ?: defaultNormalizationConfig.mergeSlashes,
pathWithEscapedSlashesAction = proto.field("path_with_escaped_slashes_action")?.stringValue ?: defaultNormalizationConfig.pathWithEscapedSlashesAction
pathWithEscapedSlashesAction = proto.field("path_with_escaped_slashes_action")?.stringValue
?: defaultNormalizationConfig.pathWithEscapedSlashesAction
)
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,9 @@ class HttpConnectionManagerFactory(
.setDelayedCloseTimeout(Duration.newBuilder().setSeconds(0).build())
.setCommonHttpProtocolOptions(httpProtocolOptions)
.setNormalizePath(BoolValue.newBuilder().setValue(normalizationConfig.normalizationEnabled).build())
.setPathWithEscapedSlashesAction(normalizationConfig.pathWithEscapedSlashesAction.toPathWithEscapedSlashesActionEnum())
.setPathWithEscapedSlashesAction(
normalizationConfig.pathWithEscapedSlashesAction.toPathWithEscapedSlashesActionEnum()
)
.setMergeSlashes(normalizationConfig.mergeSlashes)
.setCodecType(HttpConnectionManager.CodecType.AUTO)
.setHttpProtocolOptions(ingressHttp1ProtocolOptions(group.serviceName))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,6 @@ fun pathNormalizationProto(
}
}


fun accessLogBooleanFilterProto(value: Boolean? = null, fieldName: String): Value = struct {
when {
value != null -> putFields(fieldName, boolean(value))
Expand Down

0 comments on commit 2968fb2

Please sign in to comment.