You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dubbo.apache.org by fa...@apache.org on 2019/11/24 03:09:41 UTC
[dubbo-go] branch develop updated: Fix BUG: The generic fliter and
default reference filter should have ', ' seperator
This is an automated email from the ASF dual-hosted git repository.
fangyc pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/dubbo-go.git
The following commit(s) were added to refs/heads/develop by this push:
new a420214 Fix BUG: The generic fliter and default reference filter should have ',' seperator
new 71552be Merge pull request #260 from flycash/bug/GenericMergeFilter
a420214 is described below
commit a4202141c29b71fbb0c06af97674ff02a9a9b88b
Author: Ming Deng <mi...@ebay.com>
AuthorDate: Sat Nov 23 23:49:28 2019 +0800
Fix BUG: The generic fliter and default reference filter should have ',' seperator
---
config/reference_config.go | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/config/reference_config.go b/config/reference_config.go
index c63ac2e..8703c45 100644
--- a/config/reference_config.go
+++ b/config/reference_config.go
@@ -183,7 +183,7 @@ func (refconfig *ReferenceConfig) getUrlMap() url.Values {
//filter
var defaultReferenceFilter = constant.DEFAULT_REFERENCE_FILTERS
if refconfig.Generic {
- defaultReferenceFilter = constant.GENERIC_REFERENCE_FILTERS + defaultReferenceFilter
+ defaultReferenceFilter = constant.GENERIC_REFERENCE_FILTERS + "," + defaultReferenceFilter
}
urlMap.Set(constant.REFERENCE_FILTER_KEY, mergeValue(consumerConfig.Filter, refconfig.Filter, defaultReferenceFilter))