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

[pulsar-broker] add broker config to enforce producer to publish encrypted message #8055

Merged
merged 1 commit into from Oct 29, 2020

Conversation

rdhabalia
Copy link
Contributor

Motivation

As a user we need an environment to enforce producer to publish only encrypted message. therefore, adding broker configuration to enforce message encryption for broker.

@rdhabalia rdhabalia added this to the 2.7.0 milestone Sep 14, 2020
@rdhabalia rdhabalia self-assigned this Sep 14, 2020
@rdhabalia
Copy link
Contributor Author

/pulsarbot run-failure-checks

Copy link
Contributor

@hrsakai hrsakai left a comment

Choose a reason for hiding this comment

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

LGTM

@zymap
Copy link
Member

zymap commented Sep 15, 2020

Please rebase to the latest master. There have some fixes about the CI.

@BewareMyPower
Copy link
Contributor

I've fixed the testIndividualAck cpp unit test failure in #8141 , please rebase and commit again, thx!

@rdhabalia
Copy link
Contributor Author

/pulsarbot run-failure-checks

@rdhabalia
Copy link
Contributor Author

/pulsarbot run-failure-checks

@rdhabalia rdhabalia merged commit da61b56 into apache:master Oct 29, 2020
@rdhabalia rdhabalia deleted the env_broker branch October 29, 2020 19:34
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Nov 13, 2020
flowchartsman pushed a commit to flowchartsman/pulsar that referenced this pull request Nov 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants