Commit 3d69ad1
src: make Sec-WebSocket-Key check case-insensitive
Current case sensitive comparison is breaking netty-based WS clients.
replace strncmp with strncasecmp
Fixes: #7247
PR-URL: #7248
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>1 parent 88b2aa3 commit 3d69ad1Copy full SHA for 3d69ad1
File tree
Expand file treeCollapse file tree
4 files changed
+34
-3
lines changedOpen diff view settings
Filter options
- src
- test/cctest
Expand file treeCollapse file tree
4 files changed
+34
-3
lines changedOpen diff view settings
Collapse file
+6-3Lines changed: 6 additions & 3 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
1 | 1 | |
| 2 | + |
| 3 | + |
2 | 4 | |
3 | 5 | |
4 | 6 | |
| ||
445 | 447 | |
446 | 448 | |
447 | 449 | |
448 | | - |
449 | | - |
450 | | - |
| 450 | + |
| 451 | + |
| 452 | + |
| 453 | + |
451 | 454 | |
452 | 455 | |
453 | 456 | |
|
Collapse file
+10Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
219 | 219 | |
220 | 220 | |
221 | 221 | |
| 222 | + |
| 223 | + |
| 224 | + |
| 225 | + |
| 226 | + |
| 227 | + |
| 228 | + |
| 229 | + |
| 230 | + |
| 231 | + |
222 | 232 | |
223 | 233 | |
224 | 234 | |
|
Collapse file
+3Lines changed: 3 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
206 | 206 | |
207 | 207 | |
208 | 208 | |
| 209 | + |
| 210 | + |
| 211 | + |
209 | 212 | |
210 | 213 | |
211 | 214 | |
|
Collapse file
+15Lines changed: 15 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
68 | 68 | |
69 | 69 | |
70 | 70 | |
| 71 | + |
| 72 | + |
| 73 | + |
| 74 | + |
| 75 | + |
| 76 | + |
| 77 | + |
| 78 | + |
| 79 | + |
| 80 | + |
| 81 | + |
| 82 | + |
| 83 | + |
| 84 | + |
| 85 | + |
71 | 86 | |
72 | 87 | |
73 | 88 | |
|
0 commit comments