Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DirectMessageListenerContainer Does Not Consider Authentication Failures as Fatal #1347

Closed
garyrussell opened this issue Jun 7, 2021 · 0 comments · Fixed by #1349
Closed

Comments

@garyrussell
Copy link
Contributor

possibleAuthenticationFailureFatal is only used by the SMLC; docs say it applies to both.

We might need to change it to false by default for the DMLC to avoid a behavior change.

@garyrussell garyrussell added this to the 2.3.8 milestone Jun 7, 2021
garyrussell added a commit to garyrussell/spring-amqp that referenced this issue Jun 8, 2021
Resolves spring-projects#1347

Container starts and keeps trying to connect instead of honoring
the property.
garyrussell added a commit to garyrussell/spring-amqp that referenced this issue Jun 8, 2021
Resolves spring-projects#1347

Container starts and keeps trying to connect instead of honoring
the property.

**cherry-pick to 2.2.x**
garyrussell added a commit to garyrussell/spring-amqp that referenced this issue Jun 8, 2021
Resolves spring-projects#1347

Container starts and keeps trying to connect instead of honoring
the property.

**cherry-pick to 2.2.x**
artembilan pushed a commit that referenced this issue Jun 9, 2021
Resolves #1347

Container starts and keeps trying to connect instead of honoring
the property.

**cherry-pick to 2.2.x**
artembilan pushed a commit that referenced this issue Jun 9, 2021
Resolves #1347

Container starts and keeps trying to connect instead of honoring
the property.

**cherry-pick to 2.2.x**

# Conflicts:
#	spring-rabbit/src/test/java/org/springframework/amqp/rabbit/listener/DirectMessageListenerContainerIntegrationTests.java
#	src/reference/asciidoc/amqp.adoc
garyrussell added a commit that referenced this issue Jun 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant