Query的数据顺序会乱序 about gout

[guonaihong]

发下SetQuery的入参使用方式

…

---原始邮件--- 发件人: "Hang ***@***.***> 发送时间: 2023年5月1日(周一) 上午8:34 收件人: ***@***.***>; 抄送: ***@***.***>; 主题: [guonaihong/gout] Query的数据顺序会乱序 (Issue #367) Bug Report 如题：Query的数据顺序会打乱顺序；而有些入参的验证key是根据query数据的顺序生成的。 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[guonaihong]

我理了下这个功能是可以实现(简单魔改下url.Encode函数)。有点好奇，SetQuery现在给出的值是按字典序排序过的。按道理服务端，无论是正常解析比如gin之类的框架还是参数签名，都可以正常解析。是什么特殊的需求吗？

[tangtanglove]

如果SetQuery传入的是map，肯定顺序会打乱的；map是无序的，建议增加键值对的传入方法；还有一个bug是SetHeader传入Struct，tag标记不会生效

[guonaihong]

如果SetQuery传入的是map，肯定顺序会打乱的；map是无序的，建议增加键值对的传入方法；还有一个bug是SetHeader传入Struct，tag标记不会生效

SetQuery下面是用的url.Values类型编码成query string.

从源代码可以看出是会排序的。所以比较好奇乱序的使用代码。
我用nc 模拟服务端
nc -l 8080
使用gout的客户端代码使用map传参

package main

import "github.com/guonaihong/gout"

type testQuery struct {
	C string `query:"c"`
	B string `query:"b"`
	A string `query:"a"`
}

func main() {
	var err error
	err = gout.GET(":8080").Debug(true).SetQuery(gout.H{"3": "3", "5": "5", "9": "9", "8": "8"}).Do()
	if err != nil {
		// 为了方便测试，直接panic
		panic(err)
	}

	err = gout.GET(":1234").Debug(true).SetQuery(testQuery{C: "c", B: "B", A: "A"}).Do()
	if err != nil {
		// 为了方便测试，直接panic
		panic(err)
	}
}

几次执行gout代码都是按字典序给出结果。

[guonaihong]

从我发的截图可以证实，无论是map还是struct还是slice编码之后的query string都是按字典序给出的。
你刚刚提的header tag不生效我确认下。

[guonaihong]

SetHeader只有一种情况不会生效，这个header的field不符合rfc的规定，如果要所见即所得的设置header。可以使用SetHeaderRaw这个接口。

package main

import "github.com/guonaihong/gout"

type Header struct {
	Header1 string `header:"header1"`
	Header2 string `header:"header2"`
}

func main() {
	var err error
	err = gout.GET(":8080").SetHeaderRaw(Header{Header1: "aa", Header2: "bb"}).Do()
	if err != nil {
		panic(err)
	}
}

[tangtanglove]

header的问题解决了；但是query重排序的问题最好也处理一下，可以让用户自行抉择是否需要重新排序

[guonaihong]

可以我这个版本出个SetQueryRaw接口。