Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

IPv6 support for client.py #1689

Closed
wants to merge 1 commit into from
Closed

IPv6 support for client.py #1689

wants to merge 1 commit into from

Conversation

glyph
Copy link
Contributor

@glyph glyph commented Dec 11, 2016

This is an (untested) general sketch of how to use wrapClientTLS to implement TLS over IPv6, as well as faster connections over IPv4.

This is an (untested) general sketch of how to use wrapClientTLS to implement TLS over IPv6, as well as faster connections over IPv4.
@matrixbot
Copy link
Member

Can one of the admins verify this patch?

4 similar comments
@matrixbot
Copy link
Member

Can one of the admins verify this patch?

@matrixbot
Copy link
Member

Can one of the admins verify this patch?

@matrixbot
Copy link
Member

Can one of the admins verify this patch?

@matrixbot
Copy link
Member

Can one of the admins verify this patch?

@glyph glyph mentioned this pull request Dec 11, 2016
glyph added a commit to glyph/synapse that referenced this pull request Dec 11, 2016
Similar to matrix-org#1689, but for endpoint.py
)
tlsCreator = self.policyForHTTPS.creatorForNetloc(uri.host, uri.port)
def endpoint_factory(reactor, host, port, **kw):
return wrapClientTLS(tlsCreator, HostnameEndpoint(reactor, host, port, **kw)
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing closing paren

kyrias pushed a commit to kyrias/synapse that referenced this pull request Dec 12, 2016
Similar to matrix-org#1689, but for endpoint.py
@kyrias kyrias mentioned this pull request Dec 12, 2016
kyrias added a commit to kyrias/synapse that referenced this pull request Dec 12, 2016
Signed-off-by: Johannes Löthberg <johannes@kyriasis.com>
@ara4n
Copy link
Member

ara4n commented Dec 18, 2016

@kyrias has consolidated this into #1696, and we'll try to land it from there. thanks @glyph!!

@ara4n ara4n closed this Dec 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants