Skip to content

fix gh-470, support the values of QueryMap starting with '{'#540

Merged
codefromthecrypt merged 1 commit into
OpenFeign:masterfrom
lowzj:master-QueryMap
Mar 28, 2017
Merged

fix gh-470, support the values of QueryMap starting with '{'#540
codefromthecrypt merged 1 commit into
OpenFeign:masterfrom
lowzj:master-QueryMap

Conversation

@lowzj

@lowzj lowzj commented Mar 23, 2017

Copy link
Copy Markdown
Contributor

@codefromthecrypt codefromthecrypt 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.

very minor feedback. thanks for the help

}

private static String urlEncode(Object arg) {
public static String urlEncode(Object arg) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

make package private

}

private static String urlDecode(String arg) {
public static String urlDecode(String arg) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

revert as this code isn't used in this PR

@lowzj lowzj force-pushed the master-QueryMap branch from c1cb607 to 9738c77 Compare March 23, 2017 04:41
@codefromthecrypt codefromthecrypt merged commit 29d1db5 into OpenFeign:master Mar 28, 2017
zhujf1989 pushed a commit to zhujf1989/feign that referenced this pull request Apr 25, 2017
zhujf1989 added a commit to zhujf1989/feign that referenced this pull request Apr 25, 2017
@lowzj lowzj deleted the master-QueryMap branch May 3, 2017 09:15
zhujf1989 added a commit to zhujf1989/feign that referenced this pull request May 3, 2017
codefromthecrypt pushed a commit that referenced this pull request May 6, 2017
…#555)

support the query values starting with {, when use RibbonClient。different from #540.
phymbert pushed a commit to phymbert/feign that referenced this pull request Aug 22, 2017
…OpenFeign#555)

support the query values starting with {, when use RibbonClient。different from OpenFeign#540.
velo pushed a commit that referenced this pull request Oct 7, 2024
…#555)

support the query values starting with {, when use RibbonClient。different from #540.
velo pushed a commit that referenced this pull request Oct 8, 2024
…#555)

support the query values starting with {, when use RibbonClient。different from #540.
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.

2 participants