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.

Sign up for GitHub

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

Jump to bottom

push consumer set wrong options did't stop and pull consumer did't check config #935

Closed
0daypwn opened this issue Oct 10, 2022 · 0 comments · Fixed by #936
Closed

push consumer set wrong options did't stop and pull consumer did't check config #935

0daypwn opened this issue Oct 10, 2022 · 0 comments · Fixed by #936
Labels
question Further information is requested

Comments

@0daypwn
Copy link
Contributor

0daypwn commented Oct 10, 2022

BUG REPORT

  1. Please describe the issue you observed:

    • What did you do (The steps to reproduce)?

    • What did you expect to see?
      push consumer set wrong options , stop it then return err
      pull consumer set wrong options , stop it then return err

    • What did you see instead?
      push consumer set wrong options, only printed the log and did not stop.
      pull consumer did't check config

  2. Please tell us about your environment:

    • What is your OS?

    • What is your client version?

    • What is your RocketMQ version?

  3. Other information (e.g. detailed explanation, logs, related issues, suggestions on how to fix, etc):
@ShannonDing ShannonDing added the question Further information is requested label Oct 11, 2022
0daypwn pushed a commit to 0daypwn/rocketmq-client-go that referenced this issue Oct 11, 2022
[ISSUE apache#935]fix consumer option validate
a0b81af
@0daypwn 0daypwn mentioned this issue Oct 11, 2022
Merged
5 tasks
ShannonDing pushed a commit that referenced this issue Oct 14, 2022
@0daypwn
[ISSUE #935]fix consumer option validate (#936)
b3ede3a 
Co-authored-by: wuxb02 <wuxb02@mingyuanyun.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question Further information is requested
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE #935]fix consumer option validate 0daypwn/rocketmq-client-go
2 participants
@ShannonDing @0daypwn