Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

[security] Communications between broker inherit BrokerClient configuration #1969

Conversation

gaoran10
Copy link
Contributor

(cherry picked from commit 5ff8b80)

Motivation

Communications between broker inherit BrokerClient configuration.

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@gaoran10 gaoran10 self-assigned this Jul 20, 2023
@github-actions github-actions bot added the no-need-doc This pr does not need any document label Jul 20, 2023
@gaoran10 gaoran10 added release/2.11 release/2.10.4 release/3.0 and removed no-need-doc This pr does not need any document labels Jul 20, 2023
@github-actions
Copy link

@gaoran10:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions github-actions bot added the doc-info-missing This pr needs to mark a document option in description label Jul 20, 2023
@codecov
Copy link

codecov bot commented Jul 20, 2023

Codecov Report

Merging #1969 (504e478) into master (1dda879) will increase coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1969      +/-   ##
============================================
+ Coverage     17.72%   17.74%   +0.01%     
- Complexity      751      752       +1     
============================================
  Files           195      195              
  Lines         14155    14157       +2     
  Branches       1322     1324       +2     
============================================
+ Hits           2509     2512       +3     
  Misses        11463    11463              
+ Partials        183      182       -1     
Impacted Files Coverage Δ
...native/pulsar/handlers/kop/utils/ssl/SSLUtils.java 57.22% <0.00%> (-0.67%) ⬇️

... and 1 file with indirect coverage changes

@gaoran10 gaoran10 merged commit 20e4886 into streamnative:master Jul 30, 2023
22 of 23 checks passed
Demogorgon314 pushed a commit to Demogorgon314/kop that referenced this pull request Aug 14, 2023
…ration (streamnative#1969)

(cherry picked from commit 5ff8b80)

### Motivation

Communications between broker inherit BrokerClient configuration.

Co-authored-by: Enrico Olivelli <[email protected]>
(cherry picked from commit 20e4886)
Demogorgon314 pushed a commit to Demogorgon314/kop that referenced this pull request Nov 28, 2023
…ration (streamnative#1969)

(cherry picked from commit 5ff8b80)

### Motivation

Communications between broker inherit BrokerClient configuration.

Co-authored-by: Enrico Olivelli <[email protected]>
(cherry picked from commit 20e4886)
(cherry picked from commit a25bae9)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
doc-info-missing This pr needs to mark a document option in description release/2.10.4 release/2.11 release/3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants