Skip to content

Revert generate cert file permission#1878

Merged
senthuran16 merged 1 commit intowso2:event-gateway-bug-fixes-after-0.5.0from
AnujaKalahara99:egw/kafka_ssl
May 5, 2026
Merged

Revert generate cert file permission#1878
senthuran16 merged 1 commit intowso2:event-gateway-bug-fixes-after-0.5.0from
AnujaKalahara99:egw/kafka_ssl

Conversation

@AnujaKalahara99
Copy link
Copy Markdown
Contributor

Purpose

Enable local Event Gateway Kafka TLS/SASL development and harden a few related runtime paths. Resolves N/A.

Goals

Allow the dev Kafka stack to start reliably with generated certs, support Kafka TLS/SASL config in the runtime, and keep WebSub subscription sync and compacted-topic handling correct.

Approach

Added local Kafka cert generation and compose wiring, implemented Kafka TLS/SASL config support in the broker driver, verified existing compacted topics before accepting them, and removed Kafka-specific coupling from subscription
sync code.

User stories

As a developer, I can run the local Event Gateway stack with secured Kafka and have WebSub state sync continue to work correctly.

Documentation

N/A - local dev and internal runtime behavior changes only.

Automation tests

  • Unit tests
    Focused Kafka broker-driver tests added/updated; targeted package tests passed locally.
  • Integration tests
    Not run.

Security checks

Samples

Local Docker Compose Kafka dev setup with generated TLS artifacts.

Related PRs

N/A

Test environment

Ubuntu 24.04 local dev environment.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 5, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2605a4b4-5317-40f6-ad5c-d0fba7a824dc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@senthuran16 senthuran16 merged commit f59c861 into wso2:event-gateway-bug-fixes-after-0.5.0 May 5, 2026
2 checks 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.

2 participants