Skip to content

[6.2.x] Fix FactoryFinder path resolution in Windows (#1831)#1836

Merged
cshannon merged 1 commit intoapache:activemq-6.2.xfrom
cshannon:1831-6.2.x-backport
Mar 26, 2026
Merged

[6.2.x] Fix FactoryFinder path resolution in Windows (#1831)#1836
cshannon merged 1 commit intoapache:activemq-6.2.xfrom
cshannon:1831-6.2.x-backport

Conversation

@cshannon
Copy link
Copy Markdown
Contributor

This update ensures forward slashes are alow used for classpath resolution, even on all Windows versions. Also add extra validation to prevent path separators (slashes) in factory key names. Lastly added some more tests for validation

(cherry picked from commit bedb108)

This update ensures forward slashes are alow used for classpath
resolution, even on all Windows versions. Also add extra validation
to prevent path separators (slashes) in factory key names. Lastly added
some more tests for validation

This closes apache#1830

(cherry picked from commit bedb108)
@cshannon cshannon merged commit c2f8f19 into apache:activemq-6.2.x Mar 26, 2026
@cshannon cshannon deleted the 1831-6.2.x-backport branch March 26, 2026 14:24
@cshannon cshannon changed the title Fix FactoryFinder path resolution in Windows (#1831) [6.2.x] Fix FactoryFinder path resolution in Windows (#1831) Apr 9, 2026
pburgess147 pushed a commit to peersoftware/activemq that referenced this pull request Apr 17, 2026
This update ensures forward slashes are alow used for classpath
resolution, even on all Windows versions. Also add extra validation
to prevent path separators (slashes) in factory key names. Lastly added
some more tests for validation

This closes apache#1830

(cherry picked from commit bedb108)
jgallimore pushed a commit to tomitribe/activemq that referenced this pull request Apr 17, 2026
This update ensures forward slashes are alow used for classpath
resolution, even on all Windows versions. Also add extra validation
to prevent path separators (slashes) in factory key names. Lastly added
some more tests for validation

This closes apache#1830

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant