Skip to content

Add validation for LDAP network connector URIs#2077

Merged
cshannon merged 2 commits into
apache:mainfrom
cshannon:validate-ldap-nc
Jun 8, 2026
Merged

Add validation for LDAP network connector URIs#2077
cshannon merged 2 commits into
apache:mainfrom
cshannon:validate-ldap-nc

Conversation

@cshannon

@cshannon cshannon commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Validate the URIs provided by LDAP searches do not contain any protocols from the denied list.

Validate the URIs provided by LDAP searches do not contain any protocols
from the denied list.
@cshannon cshannon requested a review from jbonofre June 8, 2026 15:48

@mattrpav mattrpav left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Good improvement to move the validator to a util class

@cshannon cshannon merged commit ac3d064 into apache:main Jun 8, 2026
10 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Apache ActiveMQ v6.2.7 Jun 8, 2026
@cshannon cshannon deleted the validate-ldap-nc branch June 8, 2026 21:38
cshannon added a commit that referenced this pull request Jun 8, 2026
Validate the URIs provided by LDAP searches do not contain any protocols
from the denied list.

(cherry picked from commit ac3d064)
cshannon added a commit that referenced this pull request Jun 8, 2026
Validate the URIs provided by LDAP searches do not contain any protocols
from the denied list.

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

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants