diff options
| author | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2019-01-28 09:56:59 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-01-28 09:56:59 +0000 |
| commit | d840019192769f900f6a1a5c768368a080e651cd (patch) | |
| tree | b0b495c61a0f69dbecd32259249274e726b968b6 /changelog.d/4487.misc | |
| parent | Merge pull request #4470 from matrix-org/erikj/require_format_version (diff) | |
| download | synapse-d840019192769f900f6a1a5c768368a080e651cd.tar.xz | |
Fix idna and ipv6 literal handling in MatrixFederationAgent (#4487)
Turns out that the library does a better job of parsing URIs than our reinvented wheel. Who knew. There are two things going on here. The first is that, unlike parse_server_name, URI.fromBytes will strip off square brackets from IPv6 literals, which means that it is valid input to ClientTLSOptionsFactory and HostnameEndpoint. The second is that we stay in `bytes` throughout (except for the argument to ClientTLSOptionsFactory), which avoids the weirdness of (sometimes) ending up with idna-encoded values being held in `unicode` variables. TBH it probably would have been ok but it made the tests fragile.
Diffstat (limited to 'changelog.d/4487.misc')
| -rw-r--r-- | changelog.d/4487.misc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/changelog.d/4487.misc b/changelog.d/4487.misc new file mode 100644 index 0000000000..79de8eb3ad --- /dev/null +++ b/changelog.d/4487.misc @@ -0,0 +1 @@ +Fix idna and ipv6 literal handling in MatrixFederationAgent |