fix(chaoxing):fix JSON parsing error in content field
#5877
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks for opening this pull request! Please check out our contributing guidelines. |
|
网页端返回的 json： {
"cataid": 123,
"cfid": 123,
"content": {
"...": "...",
"size": "31415",
"puid": "12345678",
"....": "...."
}
}手机端返回的 json： {
"cataid": 123,
"cfid": 123,
"content": {
"...": "...",
"size": 31415,
"puid": 12345678,
"....": "...."
}
}错误截图：
手机端超星上传：
|
foxxorcat
reviewed
Jan 12, 2024
drivers/chaoxing/types.go
Outdated
Comment on lines
99
to
116
| func (ios *int_str) UnmarshalJSON(data []byte) error { | ||
| // 解析为int | ||
| var intValue int | ||
| if err := json.Unmarshal(data, &intValue); err == nil { | ||
| *ios = int_str(intValue) | ||
| return nil | ||
| } | ||
| // 解析string再转int | ||
| var strValue string | ||
| if err := json.Unmarshal(data, &strValue); err == nil { | ||
| intValue, err := strconv.Atoi(strValue) | ||
| if err != nil { | ||
| return fmt.Errorf("json: cannot unmarshal data into Go struct field .list.content._ of type int_string") | ||
| } | ||
| *ios = int_str(intValue) | ||
| } | ||
| return nil | ||
| } |
There was a problem hiding this comment.
没必要这样转，json字符串和数字就双引号的区别，删除双引号就是数字
intValue, err := strconv.Atoi(string(bytes.Trim(data , "\"")))
if err != nil {
return err
}
*ios = int_str(intValue)
|
另外，手机端上传的文件没有
由于文件 |
xhofe
approved these changes
Jan 14, 2024
|
Congrats on merging your first pull request! We here at behavior bot are proud of you! |
|
哥们还是不要叫计算机菜鸟了！ formidable opponent！ |
EtherealAO
pushed a commit
to EtherealAO/alist
that referenced
this pull request
Apr 12, 2024
* fix(chaoxing):fix JSON parsing error in `content` field * fix(chaoxing): optimizing `UnmarshalJSON` implementation * fix(chaoxing): use `objectID` when is empty
EtherealAO
pushed a commit
to EtherealAO/alist
that referenced
this pull request
Apr 12, 2024
* fix(chaoxing):fix JSON parsing error in `content` field * fix(chaoxing): optimizing `UnmarshalJSON` implementation * fix(chaoxing): use `objectID` when is empty
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
手机端超星上传小组云盘和网页端上传的文件在获取文件信息（
pc/resource/getResourceList&recType=2）时返回的 json 格式不同，size和puid字段可能是 string 也可能是 int。用自定义数据类型
int_str和unmarshaler接口来解决此问题。