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

[security] broker to broker hostname verification #1983

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Demogorgon314
Copy link
Member

(cherry picked from commit 10f28df)

Motivation

broker to broker hostname verification

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)

@Demogorgon314 Demogorgon314 self-assigned this Jul 28, 2023
@github-actions github-actions bot added no-need-doc This pr does not need any document labels Jul 28, 2023
@codecov
Copy link

codecov bot commented Jul 28, 2023

Codecov Report

Merging #1983 (9dbe389) into master (e931b6d) will increase coverage by 0.00%.
Report is 2 commits behind head on master.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1983   +/-   ##
=========================================
  Coverage     17.72%   17.73%           
- Complexity      751      752    +1     
=========================================
  Files           195      195           
  Lines         14156    14167   +11     
  Branches       1322     1321    -1     
=========================================
+ Hits           2509     2512    +3     
- Misses        11464    11473    +9     
+ Partials        183      182    -1     
Files Changed Coverage Δ
...ative/pulsar/handlers/kop/KafkaRequestHandler.java 1.06% <ø> (+<0.01%) ⬆️
...ansaction/TransactionMarkerChannelInitializer.java 0.00% <0.00%> (ø)
...r/transaction/TransactionMarkerChannelManager.java 0.00% <0.00%> (ø)
...sar/handlers/kop/storage/AppendRecordsContext.java 0.00% <0.00%> (ø)
...native/pulsar/handlers/kop/utils/ssl/SSLUtils.java 57.89% <0.00%> (ø)

... and 7 files with indirect coverage changes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/security no-need-doc This pr does not need any document release/2.10.4 release/2.11 release/3.0 type/enhancement Indicates an improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants