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

feature: add business data match #207

Merged
merged 1 commit into from
Oct 17, 2021
Merged

feature: add business data match #207

merged 1 commit into from
Oct 17, 2021

Conversation

wufunc
Copy link
Contributor

@wufunc wufunc commented Oct 15, 2021

Signed-off-by: wufan wufunc@gmail.com

Describe what this PR does / why we need it

    Provides the ability to match specific business data for injection by getting business data from rpc or implementing the specific spi interface.

Does this pull request fix one issue?

   Fixes #203

Describe how you did it

blade create dubbo throwCustomException --consumer --service=com.qunar.api --version=1.0.0  --b-parms='[{
"key":"a",
"value":"a_value",
“mode”:"spi"
},
{
"key":"b",
"value":"b_value",
“mode”:"spi"
}]’

@xcaspar xcaspar self-requested a review October 17, 2021 09:56
@xcaspar xcaspar added the type/feature Category issues or prs related to feature request. label Oct 17, 2021
@xcaspar xcaspar added this to the v1.4.0 milestone Oct 17, 2021
Signed-off-by: wufan <wufunc@gmail.com>
@xcaspar xcaspar merged commit 9931317 into chaosblade-io:1.4.0-dev Oct 17, 2021
@xcaspar
Copy link
Member

xcaspar commented Nov 12, 2021

#204

@hooq77
Copy link

hooq77 commented Feb 24, 2022

请问下,如果我不想使用b-params匹配,我应该怎么传值呢?

@binbin0325
Copy link
Member

那就--b-parms 整体都不传

@hooq77
Copy link

hooq77 commented Feb 25, 2022

不行,会提示illegal json

@binbin0325
Copy link
Member

binbin0325 commented Feb 25, 2022

你用的版本不对吧,使用1.5.0,相关pr:#216

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/feature Category issues or prs related to feature request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants