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

Jump to bottom

RTCDtlsTransport, certificates optional? #340

Closed
lgrahl opened this issue Jan 10, 2016 · 4 comments
Closed

RTCDtlsTransport, certificates optional? #340

lgrahl opened this issue Jan 10, 2016 · 4 comments
Labels
1.1 PR exists

Comments

@lgrahl
Copy link
Contributor

lgrahl commented Jan 10, 2016

In example 11, several RTCDtlsTransport instances are created without passing certificates. So, is the certificates parameter in the constructor of RTCDtlsTransport optional or is the example simply incorrect?
@aboba aboba added the 1.1 label Jan 11, 2016
@aboba
Copy link
Contributor

aboba commented Jan 11, 2016

The certificates parameter is not optional, so Example 11 is incorrect.

@aboba aboba mentioned this issue Jan 11, 2016
Merged
@aboba
Copy link
Contributor

aboba commented Jan 12, 2016

Note that respec does not appear to be rendering sequence < RTCCertificate > certificates in the RTCDtlsTransport constructor. This does appear legal in WebIDL, so it looks like a respec issue rather than a spec problem.

@lgrahl
Copy link
Contributor Author

lgrahl commented Jan 13, 2016

Thanks for the clarification.

@aboba
Copy link
Contributor

aboba commented Jan 14, 2016

The problem with Example 11 was fixed by PR 341.

@aboba aboba closed this as completed Jan 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1.1 PR exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aboba @lgrahl