Skip to content

fix(admin): check username for updating consumer#4756

Merged
tokers merged 5 commits into
apache:masterfrom
okaybase:patch-consumer
Aug 5, 2021
Merged

fix(admin): check username for updating consumer#4756
tokers merged 5 commits into
apache:masterfrom
okaybase:patch-consumer

Conversation

@okaybase

@okaybase okaybase commented Aug 5, 2021

Copy link
Copy Markdown
Member

What this PR does / why we need it:

fix #4680

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

@spacewander spacewander changed the title fix: check username for updating consumer fix(admin): check username for updating consumer Aug 5, 2021
Comment thread t/admin/consumers2.t Outdated
Comment thread t/admin/consumers2.t Outdated
@tokers
tokers merged commit 76f53e3 into apache:master Aug 5, 2021
@okaybase
okaybase deleted the patch-consumer branch August 5, 2021 11:50
@spacewander spacewander mentioned this pull request Aug 6, 2021
4 tasks
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.

bug: It should return 400 when update consumer with mismatched username in path and body

3 participants