[web-pubsub-express] fix Event Handler erroneous behavior due to ce-hub header not preserving case (Azure#22136)

gfortaine · web-flow · commit 7110b07dddb3 · 2022-06-08T12:53:11.000-07:00
Compare the Hub name in a case insensitive fashion.
diff --git a/sdk/web-pubsub/web-pubsub-express/src/cloudEventsDispatcher.ts b/sdk/web-pubsub/web-pubsub-express/src/cloudEventsDispatcher.ts
@@ -241,7 +241,7 @@ export class CloudEventsDispatcher {
 
     // check if hub matches
     const hub = utils.getHttpHeader(request, "ce-hub");
-    if (hub !== this.hub) {
+    if (hub?.toUpperCase() !== this.hub.toUpperCase()) {
       return false;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -241,7 +241,7 @@ export class CloudEventsDispatcher {`
`241`	`241`
`242`	`242`	`// check if hub matches`
`243`	`243`	`const hub = utils.getHttpHeader(request, "ce-hub");`
`244`		`- if (hub !== this.hub) {`
	`244`	`+ if (hub?.toUpperCase() !== this.hub.toUpperCase()) {`
`245`	`245`	`return false;`
`246`	`246`	`}`
`247`	`247`