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

IPv6 support for client.py#1689

Closed
glyph wants to merge 1 commit intomatrix-org:masterfrom
glyph:patch-1
Closed

IPv6 support for client.py#1689
glyph wants to merge 1 commit intomatrix-org:masterfrom
glyph:patch-1

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