-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SASL IO loop does not detect disconnects on Windows #1937
Comments
..and try to persist errnos (for internal use) on receive error.
..and try to persist errnos (for internal use) on receive error.
I've seen that your fix went into 0.11.6-RC3. I tested this version today (using the pre release nuget packages for both rdkafka and the .net client). Sadly I've got to report that the issue still remains. I'd be happy to help with fixing this issue but currently I'm not able to build the solution. I alwyas get linker errors, saying that external symbols such as _ictest and _sscanf, __iob_func cannot be resolved. The source seems to be (partially?) in openssl, but I am using the version mentioned in the readme. Any directions regarding this issue? |
@stefanseufert That happens when trying to link OpenSSL (with an older msvc toolchain, 120) statically on a newer Visual Studio (toolchain 140). |
@edenhill Thank you, changing the directory made my build work (the interceptor test project still fails, but I simply excluded this for now). I'll try to pinpoint the issue (most probably this afternoon) and will keep you posted. |
The cause was even simpler than assumed in my original analysis. The poll result was not checked for the disconnect flag (POLLHUP) while in AUTH state. The pull request above adds a check for this case in the same way the situation is handled for the UP state. |
I'm also waiting for a fix to this! Thanks @stefanseufert |
Yeah, please integrate it in v0.11.6 and do not wait for version 1.0.0, it's a really nasty problem! Thx @stefanseufert |
As described in confluentinc/confluent-kafka-dotnet#285 (comment)
The text was updated successfully, but these errors were encountered: