[connector/routing] Fix configuration error with request context

Remove a residual fallthrough keyword

Fixes #37410

Added unit tests for valid request context.

Updated changelog.

CHANGELOG.md

@@ -7,6 +7,10 @@ If you are looking for developer-facing changes, check out [CHANGELOG-API.md](./
 
 <!-- next version -->
 
+### 🧰 Bug fixes 🧰
+
+- `routingconnector`: Fix config validation when using request context. (#37410)
+
 ## v0.118.0
 
 ### 🛑 Breaking changes 🛑

connector/routingconnector/config.go

@@ -76,7 +76,6 @@ func (c *Config) Validate() error {
 			if _, err := parseRequestCondition(item.Condition); err != nil {
 				return err
 			}
-			fallthrough
 		default:
 			return errors.New("invalid context: " + item.Context)
 		}

connector/routingconnector/config_test.go

@@ -233,6 +233,20 @@ func TestValidateConfig(t *testing.T) {
 			},
 			error: `"request" context requires a 'condition'`,
 		},
+		{
+			name: "request context with condition",
+			config: &Config{
+				Table: []RoutingTableItem{
+					{
+						Context:   "request",
+						Condition: `request["attr"] == "acme"`,
+						Pipelines: []pipeline.ID{
+							pipeline.NewIDWithName(pipeline.SignalTraces, "otlp"),
+						},
+					},
+				},
+			},
+		},
 		{
 			name: "request context with invalid condition",
 			config: &Config{