Skip to content

Conversation

@ArtyomSavchenko
Copy link
Member

No description provided.

Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces Kafka message compression using GZIP and fixes a null pointer exception in the FindSecurityMiddleware's findAll method.

  • Adds GZIP compression to Kafka producers to reduce network bandwidth
  • Fixes exception when options parameter is null in FindSecurityMiddleware.findAll
  • Bumps versions for both @hcengineering/middleware (0.7.19 → 0.7.20) and @hcengineering/kafka (0.7.16 → 0.7.17)

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/kafka/src/index.ts Adds CompressionTypes import and enables GZIP compression for Kafka producer
packages/kafka/package.json Version bump to 0.7.17
packages/kafka/CHANGELOG.md Documents compression feature
packages/kafka/CHANGELOG.json Structured changelog entry for compression
packages/middleware/src/findSecurity.ts Changes null check from !== undefined to != null to prevent NPE on destructuring
packages/middleware/package.json Version bump to 0.7.20
packages/middleware/CHANGELOG.md Documents findAll fix
packages/middleware/CHANGELOG.json Structured changelog entry for fix
common/changes/@hcengineering/middleware/fix-formatting_2025-11-12-14-59.json Removes change file after incorporation into changelog

Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
@ArtyomSavchenko ArtyomSavchenko merged commit 77086c3 into main Nov 19, 2025
1 check passed
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