#1120 Stream encrypted not reported for direct TLS connections
Reporter
Zash
Owner
Zash
Created
Updated
Stars
★ (1)
Tags
Status-Fixed
Milestone-0.12
Type-Defect
Priority-Medium
Difficulty-Easy
Zash
on
What steps will reproduce the problem?
1. Have both direct TLS and starttls enabled.
2. Observe connections
What is the expected output?
Same "Stream encrypted" message
What do you see instead?
Only starttls connections get that message
What version of the product are you using? On what operating system?
The starttls-only logging was added in 969e0a054795, included in 0.10.0
Please provide any additional information below.
Relevant code is in `mod_c2s`
Direct TLS connections have code in `listener.onconnect` and normal connections have code in `stream_callbacks.streamopend` for inspecting new TLS properties.
Reported by Wiktor in the chatroom.
What steps will reproduce the problem? 1. Have both direct TLS and starttls enabled. 2. Observe connections What is the expected output? Same "Stream encrypted" message What do you see instead? Only starttls connections get that message What version of the product are you using? On what operating system? The starttls-only logging was added in 969e0a054795, included in 0.10.0 Please provide any additional information below. Relevant code is in `mod_c2s` Direct TLS connections have code in `listener.onconnect` and normal connections have code in `stream_callbacks.streamopend` for inspecting new TLS properties. Reported by Wiktor in the chatroom.
Fixed in https://hg.prosody.im/trunk/rev/d629135450ec
Changes