Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[webtransport] Set ENABLE_CONNECT_PROTOCOL settings flag #31573

Merged
merged 1 commit into from
Nov 17, 2021

Conversation

bashi
Copy link
Member

@bashi bashi commented Nov 10, 2021

This is required to use an Extended CONNECT method. WebTransport
uses an Extended CONNECT method.

This is required to use an Extended CONNECT method. WebTransport
uses an Extended CONNECT method.
Copy link
Contributor

@yutakahirano yutakahirano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@bashi bashi closed this Nov 10, 2021
@bashi bashi reopened this Nov 10, 2021
@bashi bashi closed this Nov 10, 2021
@bashi bashi reopened this Nov 10, 2021
@bashi
Copy link
Member Author

bashi commented Nov 15, 2021

The CI error doesn't seem to be specific to this PR. cd docs; make html fails with the same error on my machine.

$ make html
# ...
Running Sphinx v2.4.4

Warning, treated as error:
node class 'meta' is already registered, its visitors will be overridden
make: *** [Makefile:24: html] Error 2
rm tools/webtransport tools/wptserve tools/third_party/pywebsocket3 tools/wptrunner tools/certs
zsh: exit 2     make html

Maybe we should update sphinx version or specify docutil version?
sphinx-doc/sphinx#9841

@foolip @jgraham Any suggestion?

@bashi
Copy link
Member Author

bashi commented Nov 17, 2021

@foolip Friendly ping. Could you give advice on how to merge this PR?

@foolip
Copy link
Member

foolip commented Nov 17, 2021

The documentation failure is pre-existing, confirmed yesterday by #28759.

@jgraham I know you were looking at docs recently, is there an open issue for this?

For this PR, I'll just go ahead and merge.

@foolip foolip merged commit 0892b70 into web-platform-tests:master Nov 17, 2021
@bashi
Copy link
Member Author

bashi commented Nov 18, 2021

Thank you for merging!

Gabisampaio pushed a commit to Gabisampaio/wpt that referenced this pull request Nov 18, 2021
…m-tests#31573)

This is required to use an Extended CONNECT method. WebTransport
uses an Extended CONNECT method.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants