Skip to content
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.

Sign up for GitHub

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

Jump to bottom

When SSL_get_error() returns SSL_ERROR_SYSCALL, include the error from GetLastError() #14

Open
rlebeau opened this issue Dec 14, 2023 · 0 comments
Open

When SSL_get_error() returns SSL_ERROR_SYSCALL, include the error from GetLastError() #14

rlebeau opened this issue Dec 14, 2023 · 0 comments
Labels
Element: SSL/TLS Issues related to SSL/TLS handling, TIdSSLIOHandlerSocketBase and descendants Status: Review Needed Issue needs further review to decide next status Type: Enhancement Issue is proposing a new feature/enhancement

Comments

@rlebeau
Copy link
Member

rlebeau commented Dec 14, 2023

BTW, when SSL_get_error returns 5 (SSL_ERROR_SYSCALL), It would be wise to include actual message from SysErrorMessage(GetLastError) that was in this case returning 'An operation was attempted on something that is not a socket". Raising 'Failed to shutdown the TLS connection.', doesn't give much clues what happened.

Originally posted by @rapa4362 in IndySockets/Indy#513 (comment)
@rlebeau rlebeau added Type: Enhancement Issue is proposing a new feature/enhancement Element: SSL/TLS Issues related to SSL/TLS handling, TIdSSLIOHandlerSocketBase and descendants labels Dec 14, 2023
@rlebeau rlebeau added the Status: Review Needed Issue needs further review to decide next status label Jan 17, 2024
@rlebeau rlebeau transferred this issue from IndySockets/Indy Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Element: SSL/TLS Issues related to SSL/TLS handling, TIdSSLIOHandlerSocketBase and descendants Status: Review Needed Issue needs further review to decide next status Type: Enhancement Issue is proposing a new feature/enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rlebeau