Skip to content

[ISSUE #424] Fix NPE when using ExtRocketMQTemplateConfiguration regi…#433

Open
ljznb666 wants to merge 4 commits into
apache:masterfrom
ljznb666:hotfix/issue_424_v2
Open

[ISSUE #424] Fix NPE when using ExtRocketMQTemplateConfiguration regi…#433
ljznb666 wants to merge 4 commits into
apache:masterfrom
ljznb666:hotfix/issue_424_v2

Conversation

@ljznb666

Copy link
Copy Markdown

Fix NPE when using ExtRocketMQTemplateConfiguration register RocketMQTemplate

@panzhi33

Copy link
Copy Markdown
Contributor

compilation fails, do not write Chinese comments

@ljznb666

Copy link
Copy Markdown
Author

compilation fails, do not write Chinese comments

I have revised the content of the comments

@panzhi33

panzhi33 commented Aug 8, 2022

Copy link
Copy Markdown
Contributor

image
the new java class need add license,like this

@panzhi33

Copy link
Copy Markdown
Contributor

@lijunznb666 ping

@RockteMQ-AI RockteMQ-AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review: Approved ✅

PR: #433 — [ISSUE #424] Fix NPE when using ExtRocketMQTemplateConfiguration
Type: Bug fix (3 files, +120/-17)

Assessment

Fixes NullPointerException during ExtRocketMQTemplate registration. Refactors ExtProducerResetConfiguration for safer bean initialization. Includes sample code.

Verdict

✅ Correct NPE fix with proper refactoring.


🤖 Automated review by oss-sentinel-ai

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants