Skip to content

Conversation

@Cizzle
Copy link
Member

@Cizzle Cizzle commented Dec 18, 2017

Ignores the close notify warnings which are expected to end SSL connections and ignores false errors that appear when not enough data is present yet for non-blocking sockets.

@Cizzle Cizzle added this to the v1.8.3 milestone Dec 18, 2017
@vanosg
Copy link
Member

vanosg commented Dec 19, 2017

On a leaf bot with mkcobxs enabled on console, I got none of the 'usual' noise.

Adding the +d flag gives me stuff like

[03:35:00] TLS: state change: unknown state
[03:35:00] TLS: state change: unknown state
[03:35:00] TLS: state change: unknown state
[03:35:00] TLS: state change: unknown state
[03:35:00] TLS: awaiting more reads
[03:35:00] TLS: awaiting more reads
[03:35:00] sockread EAGAIN: 10 11 (Resource temporarily unavailable)
[03:35:00] TLS: awaiting more reads
[03:35:00] dequeue_sockets(): errno = 11 (Resource temporarily unavailable) on 10
--[03:35:00] TLS: state change: unknown state
[03:35:00] TLS: state change: unknown state

which seems like a totally appropriate place to stash those messages.

LGTM

@vanosg vanosg merged commit d12d180 into develop Dec 19, 2017
@vanosg vanosg deleted the task/sslup branch January 21, 2018 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants