envoyproxy / envoyproxy/envoy

Add a route name field to each route in route.Route list (HTTP route)

Open
#6,364 4 comments 0 reactions 1 assignee Claimed by @drao9 View on GitHub
enhancement help wanted
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 22h
Merged PRs (30d)
430

Description

**Issue Template**

*Title*: Add a route name field to each route in route.Route list (HTTP route)

*Description*:
This is a feature request for adding a name field to the http routes. It would be enabled by default but if the name is not specified in the config, then it won't be enabled. This feature has helped us a lot with logging and debugging at eBay. One of the use cases is when there are 100s of routes and we want to check which route has been matched; the name makes it easier to identify. We wanted to see if this will be useful for the community. We have a PR that can be made for this case. If there is interest, I can also make a subsequent PR adding the route name field to access logs. Here is an example of how the config JSON for the route.Route object would look like:

![Screen Shot 2019-03-22 at 4 19 36 PM](https://user-images.githubusercontent.com/12895455/54857962-5e88d100-4cbf-11e9-99af-40218b872e36.png)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.