Skip to content

Conversation

@ronething
Copy link
Contributor

Description

Fixes # (issue)

The data plane did not properly handle encryption and decryption when configuring a non authentication plugin on a consumer. This bug resulted in error logs and lack of encryption of sensitive information about the configuration of such plugins in etcd.

error log like:

[lua] plugin.lua:1027: decrypt_conf(): failed to get schema for plugin: csrf, context: ngx.timer"

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

Signed-off-by: ashing <axingfly@gmail.com>
Signed-off-by: ashing <axingfly@gmail.com>
Signed-off-by: ashing <axingfly@gmail.com>
@ronething ronething marked this pull request as ready for review September 24, 2024 09:22
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Sep 24, 2024
@dosubot dosubot bot added the bug Something isn't working label Sep 24, 2024
@ronething ronething merged commit d38d5b6 into apache:master Sep 25, 2024
31 checks passed
@ronething ronething deleted the fix/none_auth_plugin_in_consumer branch September 25, 2024 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants