-
-
Notifications
You must be signed in to change notification settings - Fork 653
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
OutboundTLS/plugin-redis missing logerror #3324
Comments
Hmm there is loginfo too right after connection. I think must have something terribly wrong with my stack, I am seeing errors on code which previously worked without major changes :) |
I am new, what is OutboundTLS? |
Restore line 20 (from my open PR) in that file and I think you'll find it better. Also, share a couple log entries, I think they'll be sub-par format, so I might add loggers to OBTLS differently. |
should be fixed in #3322 |
Thanks! logger.add_log_methods(this) work, PR will test later
loginfo at https://github.com/haraka/haraka-plugin-redis/blob/ca6c766ea19339748b59991d89c882f5c7fe1b2d/index.js#L187 not available and therefore throwing exception catched later and printed with again not available logerror... |
As far as I understand, OutboundTLS is not a plugin, it doesn't have the logging capabilities that the Redis plugin depends on so it fails spectacularly in this edge case
The text was updated successfully, but these errors were encountered: