Skip to content

Conversation

@eddumelendez
Copy link
Contributor

Currently, if CachingConnectionFactory is excluded no ConnectionFactory is
created.

@eddumelendez
Copy link
Contributor Author

eddumelendez commented Aug 6, 2023

PR #17531 fixed it in ActiveMQ

…is excluded

Currently, if `CachingConnectionFactory` is excluded no ConnectionFactory is
created.
@eddumelendez eddumelendez force-pushed the fix_artemis_autoconfiguration branch from 76c2271 to a5b7460 Compare August 6, 2023 04:17
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 6, 2023
@snicoll snicoll added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 7, 2023
@snicoll snicoll self-assigned this Aug 7, 2023
@snicoll snicoll added this to the 2.7.15 milestone Aug 7, 2023
@snicoll snicoll changed the title Fix ActiveMQConnectionFactory creation when CachingConnectionFactory is excluded ArtemisConnectionFactory is not configured when CachingConnectionFactory is missing and enabled properties are false Aug 7, 2023
@snicoll snicoll changed the title ArtemisConnectionFactory is not configured when CachingConnectionFactory is missing and enabled properties are false Artemis ConnectionFactory is not configured when CachingConnectionFactory is missing and enabled properties are false Aug 7, 2023
snicoll pushed a commit that referenced this pull request Aug 7, 2023
Previously, if spring-jms is not available and a non-pool connection
factory is requested (the default), no `CachingConnectionFactory` is
created. This commit makes sure to separate the two conditions, so
that a connection factory can be obtained even if spring-jms is not
on the classpath.

See gh-36767
@snicoll snicoll closed this in 6ccc13f Aug 7, 2023
@snicoll
Copy link
Member

snicoll commented Aug 7, 2023

Good catch, thanks @eddumelendez!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants