Skip to content

NIFI-11848 Allocate unused port in TlsCertificateAuthorityTest#7514

Closed
nandorsoma wants to merge 1 commit intoapache:mainfrom
nandorsoma:NIFI-11848
Closed

NIFI-11848 Allocate unused port in TlsCertificateAuthorityTest#7514
nandorsoma wants to merge 1 commit intoapache:mainfrom
nandorsoma:NIFI-11848

Conversation

@nandorsoma
Copy link
Copy Markdown
Contributor

@nandorsoma nandorsoma commented Jul 22, 2023

Summary

NIFI-11848

Tracking

Please complete the following tracking steps prior to pull request creation.

Issue Tracking

Pull Request Tracking

  • Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
  • Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

  • Pull Request based on current revision of the main branch
  • Pull Request refers to a feature branch with one commit containing changes

Verification

Please indicate the verification steps performed prior to pull request creation.

Build

  • Build completed using mvn clean install -P contrib-check
    • JDK 17

Licensing

  • New dependencies are compatible with the Apache License 2.0 according to the License Policy
  • New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

  • Documentation formatting appears as expected in rendered files

exceptionfactory
exceptionfactory approved these changes Jul 24, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@exceptionfactory exceptionfactory left a comment

Choose a reason for hiding this comment

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

Thanks for the improvement @nandorsoma, looks good! +1 merging

exceptionfactory pushed a commit that referenced this pull request Jul 24, 2023
@nandorsoma @exceptionfactory
NIFI-11848 Allocate unused port in TlsCertificateAuthorityTest
d45ccf5 
This closes #7514

Signed-off-by: David Handermann <exceptionfactory@apache.org>

(cherry picked from commit 84b4d10)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@exceptionfactory exceptionfactory exceptionfactory approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nandorsoma @exceptionfactory