Google Git
Sign in
chromium / chromium / src / ebf43c3172cc0640f79a05b13d225a4caf126db0
commitebf43c3172cc0640f79a05b13d225a4caf126db0[log] [tgz]
authorDavid Benjamin <davidben@chromium.org>Fri May 19 21:21:52 2023
committerChromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com>Fri May 19 21:21:52 2023
treece919b8b842f9ffb8d4bc96da3ac2b80e1b738bc
parent8baead03b8075cca68d42c2212f638fb363e40e5 [diff]
Handle alternative endpoints in HTTP/2 and HTTP/3 cross-name pooling check

Treat service endpoints as eligible for cross-name pooling if they would
have been eligible for the ALPN protocol we're accessing. Additionally,
if we have no prior knowledge of a host supporting HTTP/3 (i.e. no
Alt-Svc), cross-name pooling requires the HTTP/3 in HTTPS-RR, to align
with the behavior on a new connection.

(Linked this to crbug.com/1264933 because that's where the TODO is
associated, but this is mostly post-HTTPS-RR cleanup and not directly
ECH-related.)

This allows us to remove the GetNonProtocolEndpoints helper function.

Bug: 1264933, 126134
Change-Id: If121b0e759b5e530def76057935f18861e3d064a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4516075
Reviewed-by: Tsuyoshi Horo <horo@chromium.org>
Commit-Queue: David Benjamin <davidben@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1146719}
6 files changed
tree: ce919b8b842f9ffb8d4bc96da3ac2b80e1b738bc
  1. android_webview/
  2. apps/
  3. ash/
  4. base/
  5. build/
  6. build_overrides/
  7. buildtools/
  8. cc/
  9. chrome/
  10. chromecast/
  11. chromeos/
  12. codelabs/
  13. components/
  14. content/
  15. courgette/
  16. crypto/
  17. dbus/
  18. device/
  19. docs/
  20. extensions/
  21. fuchsia_web/
  22. gin/
  23. google_apis/
  24. google_update/
  25. gpu/
  26. headless/
  27. infra/
  28. ios/
  29. ipc/
  30. media/
  31. mojo/
  32. native_client_sdk/
  33. net/
  34. pdf/
  35. ppapi/
  36. printing/
  37. remoting/
  38. rlz/
  39. sandbox/
  40. services/
  41. skia/
  42. sql/
  43. storage/
  44. styleguide/
  45. testing/
  46. third_party/
  47. tools/
  48. ui/
  49. url/
  50. weblayer/
  51. .clang-format
  52. .clang-tidy
  53. .eslintrc.js
  54. .git-blame-ignore-revs
  55. .gitattributes
  56. .gitignore
  57. .gn
  58. .mailmap
  59. .rustfmt.toml
  60. .vpython3
  61. .yapfignore
  62. ATL_OWNERS
  63. AUTHORS
  64. BUILD.gn
  65. CODE_OF_CONDUCT.md
  66. codereview.settings
  67. DEPS
  68. DIR_METADATA
  69. LICENSE
  70. LICENSE.chromium_os
  71. OWNERS
  72. PRESUBMIT.py
  73. PRESUBMIT_test.py
  74. PRESUBMIT_test_mocks.py
  75. README.md
  76. WATCHLISTS
README.md

Logo Chromium

Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.

The project's web site is https://www.chromium.org.

To check out the source code locally, don't use git clone! Instead, follow the instructions on how to get the code.

Documentation in the source is rooted in docs/README.md.

Learn how to Get Around the Chromium Source Code Directory Structure .

For historical reasons, there are some small top level directories. Now the guidance is that new top level directories are for product (e.g. Chrome, Android WebView, Ash). Even if these products have multiple executables, the code should be in subdirectories of the product.

If you found a bug, please file it at https://crbug.com/new.