What happened: proto中定义optional的字段，当前端传的query string的字段是空字符串时，后端解

Hi, <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

proto. When the field of the query string passed by the front end is an empty string, the field parsed by the back end is nil. about kratos HOT 3 CLOSED

jeanh0w commented on May 28, 2024

proto. When the field of the query string passed by the front end is an empty string, the field parsed by the back end is nil.

from kratos.

Comments (3)

dosubot commented on May 28, 2024 3

🤖

感谢您的反馈。我看到您在使用Kratos v2.7.1版本，但是最新的版本是v2.7.2。我想知道您是否尝试过在最新版本上复现这个问题？

另外，我需要更多的信息来帮助我理解这个问题。您能提供一些关于您的环境的更多信息吗？例如，您正在使用的操作系统和版本，以及您的Go环境配置等。

最后，如果可能的话，能否提供一些更详细的代码示例，以便我更好地理解您的问题？这将有助于我更准确地找到问题的原因。

期待您的回复。

This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.

Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.

If you want to continue the conversation, start your reply with @dosu-bot.

from kratos.

dosubot commented on May 28, 2024 2

Hi, @jeanh0w,

I'm helping the Kratos team manage their backlog and am marking this issue as stale. The issue concerns the back end parsing an empty string as nil instead of an empty string in Kratos version v2.7.1 and Go version go1.21.1. There have been discussions about checking if the issue persists in the latest version, v2.7.2, and requests for more information about the user's environment. Additionally, there are questions about a specific code change that may have caused the issue and links to the relevant code sections for further investigation.

Could you please confirm if this issue is still relevant to the latest version of the Kratos repository? If it is, please let the Kratos team know by commenting on the issue. Otherwise, feel free to close the issue yourself, or the issue will be automatically closed in 7 days.

Thank you for your understanding and cooperation. If you have any further questions or updates, feel free to reach out.

from kratos.

kvii commented on May 28, 2024

@tonybase Why did you add a if value == "" { return nil } is this #2848? It caused this issue. changes detail

See the code below, should we return errors in parseField when "fd" is a non-string kind and value is ""?

kratos/encoding/form/proto_decode.go

Lines 116 to 126 in c7fa51d

 func populateField(fd protoreflect.FieldDescriptor, v protoreflect.Message, value string) error { 

 if value == "" { 

 return nil 

 } 

 val, err := parseField(fd, value) 

 if err != nil { 

 return fmt.Errorf("parsing field %q: %w", fd.FullName().Name(), err) 

 } 

 v.Set(fd, val) 

 return nil 

 }

kratos/encoding/form/proto_decode.go

Lines 161 to 168 in c7fa51d

 func parseField(fd protoreflect.FieldDescriptor, value string) (protoreflect.Value, error) { 

 switch fd.Kind() { 

 case protoreflect.BoolKind: 

 v, err := strconv.ParseBool(value) 

 if err != nil { 

 return protoreflect.Value{}, err 

 } 

 return protoreflect.ValueOfBool(v), nil

from kratos.

proto. When the field of the query string passed by the front end is an empty string, the field parsed by the back end is nil. about kratos HOT 3 CLOSED

Comments (3)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	func populateField(fd protoreflect.FieldDescriptor, v protoreflect.Message, value string) error {
	if value == "" {
	return nil
	}
	val, err := parseField(fd, value)
	if err != nil {
	return fmt.Errorf("parsing field %q: %w", fd.FullName().Name(), err)
	}
	v.Set(fd, val)
	return nil
	}

	func parseField(fd protoreflect.FieldDescriptor, value string) (protoreflect.Value, error) {
	switch fd.Kind() {
	case protoreflect.BoolKind:
	v, err := strconv.ParseBool(value)
	if err != nil {
	return protoreflect.Value{}, err
	}
	return protoreflect.ValueOfBool(v), nil