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

添加了基于 SCRAM-SHA512 加密方式的连接 #98

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

wuxi889
Copy link

@wuxi889 wuxi889 commented Feb 20, 2024

Added

  • 添加了基于 SCRAM-SHA512 加密方式的连接;

前置条件为需要开启 ssl,使用时 config/autoload/kafka.php 文件的 sasl 配置为

...
'sasl' => [
    'type' => \longlang\phpkafka\Sasl\ScramSha512Sasl::class,
    'username' => env('KAFKA_SASL_USERNAME', ''),
    'password' => env('KAFKA_SASL_PASSWORD', ''),
    // 是否验证第二次握手的服务器响应消息的签名
    'verify_final_signature' => (bool) env('KAFKA_SASL_VERIFY_FINAL_SIGNATURE', true),
],
...

Changed

  • 调整心跳检测过程中获取上次检测时间为 0 时会抛出异常的问题;

@wuxi889
Copy link
Author

wuxi889 commented Feb 21, 2024

@Yurunsoft @limingxinleo

@Yurunsoft
Copy link
Member

有 2 个测试没有通过,建议先解决一下

@wuxi889
Copy link
Author

wuxi889 commented Feb 21, 2024

有 2 个测试没有通过,建议先解决一下

@Yurunsoft phpstan 我在本地(src/Sasl)未测出问题,麻烦告知下线上的测试环境。
image

还有 php-cs-fix 也是,这个错误有点奇怪
image

@wuxi889
Copy link
Author

wuxi889 commented Aug 29, 2024

image
image
image

php cs 与 php stan 已测试通过 @Yurunsoft @limingxinleo

@PedroPMS
Copy link

Any updates? Im tying to use SCRAM-SHA512 on producer too.

@wuxi889
Copy link
Author

wuxi889 commented Sep 14, 2024

Any updates? Im tying to use SCRAM-SHA512 on producer too.

Apologies, the author hasn't merged this MR yet. In the meantime, you can fork the project, modify the composer.json to point to your fork in the repositories section, and use it temporarily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants