Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

RdKafkaConsumer crash #1357

Open
alexshiloov opened this issue Aug 26, 2024 · 0 comments
Open

RdKafkaConsumer crash #1357

alexshiloov opened this issue Aug 26, 2024 · 0 comments

Comments

@alexshiloov
Copy link

175 line https://github.com/php-enqueue/rdkafka/blob/master/RdKafkaConsumer.php#L175

$message = $this->serializer->toMessage($kafkaMessage->payload);

It will lead to crash, if someone push to kafka empty msg for example.
I think it is not correct for crashing, maybe skipping this msg or returning null

@github-staff github-staff deleted a comment from Lxx-c Oct 23, 2024
@github-staff github-staff deleted a comment from Lxx-c Oct 23, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant