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.

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

fix(plugins): change log level from error to info when restarting external plugin instance #8652

Merged
merged 3 commits into from
May 25, 2022

Conversation

ealogar
Copy link
Contributor

@ealogar ealogar commented Apr 7, 2022

Summary

When the external plugin instances (go, python, ...) loses the instances_id in a event handler, the instance is restarted to complete the plugin call but a weird error appear:

2021/08/20 10:23:59 [**error**] 30#0: *51336 [kong] mp_rpc.lua:308 [jwe] no plugin instance 0, client: 10.240.0.178, server: kong, request: "POST /example HTTP/1.1", host: "example.es"

We change the log level to info as this is not a real problem as the plugin is restarted lately in code (considering the message string as well)

@mayocream mayocream added chore Not part of the core functionality of kong, but still needed core/logs labels Apr 11, 2022
@github-actions github-actions bot added core/language/go core/language/js core/language/python and removed chore Not part of the core functionality of kong, but still needed core/logs labels Apr 11, 2022
@jschmid1 jschmid1 requested a review from mayocream April 13, 2022 12:16
@RobSerafini RobSerafini requested a review from jschmid1 May 6, 2022 21:16
@StarlightIbuki StarlightIbuki requested a review from a team as a code owner May 18, 2022 10:01
@fffonion fffonion merged commit 0d559f1 into Kong:master May 25, 2022
@fffonion
Copy link
Contributor

Thank you @ealogar !

@ealogar ealogar deleted the fix/protobuf_error_msg branch May 25, 2022 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants