Skip to content

Commit

Permalink
fix typos
Browse files Browse the repository at this point in the history
  • Loading branch information
radoslaw.chrzanowski committed Mar 10, 2025
1 parent f04810f commit 59d6e2f
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).

## [0.22.11]
### Changed
- white list for enabling separated routes for status endpoints
- whitelist for enabling separated routes for status endpoints

## [0.22.10]
### Changed
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ fun Route.ingressServiceRoute() {
.toCluster("local_service")
}

fun Route.ingresStatusRoute() {
fun Route.ingressStatusRoute() {
this.matchingOnPrefix("/status/")
.publicAccess()
.toCluster("local_service")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ import pl.allegro.tech.servicemesh.envoycontrol.groups.hasRequestHeadersToRemove
import pl.allegro.tech.servicemesh.envoycontrol.groups.hasResponseHeaderToAdd
import pl.allegro.tech.servicemesh.envoycontrol.groups.hasSingleVirtualHostThat
import pl.allegro.tech.servicemesh.envoycontrol.groups.hasStatusVirtualClusters
import pl.allegro.tech.servicemesh.envoycontrol.groups.ingresStatusRoute
import pl.allegro.tech.servicemesh.envoycontrol.groups.ingressServiceRoute
import pl.allegro.tech.servicemesh.envoycontrol.groups.ingressStatusRoute
import pl.allegro.tech.servicemesh.envoycontrol.groups.matchingOnAnyMethod
import pl.allegro.tech.servicemesh.envoycontrol.groups.matchingOnMethod
import pl.allegro.tech.servicemesh.envoycontrol.groups.matchingOnPrefix
Expand Down Expand Up @@ -175,17 +175,17 @@ internal class EnvoyIngressRoutesFactoryTest {
},
*adminRoutes,
{
ingresStatusRoute()
ingressStatusRoute()
matchingOnMethod("GET")
matchingRetryPolicy(retryPolicyProps.perHttpMethod["GET"]!!)
},
{
ingresStatusRoute()
ingressStatusRoute()
matchingOnMethod("HEAD")
matchingRetryPolicy(retryPolicyProps.perHttpMethod["HEAD"]!!)
},
{
ingresStatusRoute()
ingressStatusRoute()
matchingOnAnyMethod()
hasNoRetryPolicy()
},
Expand Down Expand Up @@ -233,7 +233,7 @@ internal class EnvoyIngressRoutesFactoryTest {

@Test
@Suppress("LongMethod")
fun `should create not create routes for status endpoints if the service is not whitelisted`() {
fun `should not create routes for status endpoints if the service is not whitelisted`() {
// given
val routesFactory = EnvoyIngressRoutesFactory(SnapshotProperties().apply {
routes.status.apply {
Expand Down Expand Up @@ -300,7 +300,7 @@ internal class EnvoyIngressRoutesFactoryTest {

@Test
@Suppress("LongMethod")
fun `should create create routes for status endpoints when whitelist contains wildcard`() {
fun `should create routes for status endpoints when whitelist contains wildcard`() {
// given
val routesFactory = EnvoyIngressRoutesFactory(SnapshotProperties().apply {
routes.status.apply {
Expand Down Expand Up @@ -346,17 +346,17 @@ internal class EnvoyIngressRoutesFactoryTest {
hasOnlyRoutesInOrder(
*adminRoutes,
{
ingresStatusRoute()
ingressStatusRoute()
matchingOnMethod("GET")
matchingRetryPolicy(retryPolicyProps.perHttpMethod["GET"]!!)
},
{
ingresStatusRoute()
ingressStatusRoute()
matchingOnMethod("HEAD")
matchingRetryPolicy(retryPolicyProps.perHttpMethod["HEAD"]!!)
},
{
ingresStatusRoute()
ingressStatusRoute()
matchingOnAnyMethod()
hasNoRetryPolicy()
},
Expand Down

0 comments on commit 59d6e2f

Please sign in to comment.