Skip to content

Commit

Permalink
Update stale comment: We use colon instead of underscore between name…
Browse files Browse the repository at this point in the history
…space and route name for uniquely identifying the route in the router
  • Loading branch information
Ravi Sankar Penta committed Aug 22, 2017
1 parent 842eedc commit 76edd6d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/router/template/router.go
Expand Up @@ -672,10 +672,10 @@ func (r *templateRouter) routeKey(route *routeapi.Route) string {
name := controller.GetSafeRouteName(route.Name)

// Namespace can contain dashes, so ${namespace}-${name} is not
// unique, use an underscore instead - ${namespace}_${name} akin
// unique, use an underscore instead - ${namespace}:${name} akin
// to the way domain keys/service records use it ala
// _$service.$proto.$name.
// Note here that underscore (_) is not a valid DNS character and
// Note here that colon (:) is not a valid DNS character and
// is just used for the key name and not for the record/route name.
// This also helps the use case for the key used as a router config
// file name.
Expand Down

0 comments on commit 76edd6d

Please sign in to comment.