fix: NewRESTAddress use http as default scheme

judehung · judehung · commit f54e88f41394 · 2025-03-10T10:00:11.000+08:00
fixes #990 The PR #986 set the scheme configurable in NewRESTAddress, but didn't consider a default value when scheme is empty string. This commit will set http as the default value when scheme is an empty string. Signed-off-by: Jude Hung <jude@iotechsys.com>
diff --git a/dtos/address.go b/dtos/address.go
@@ -69,6 +69,9 @@ type RESTAddress struct {
 }
 
 func NewRESTAddress(host string, port int, httpMethod string, scheme string) Address {
+	if scheme == "" {
+		scheme = common.HTTP
+	}
 	return Address{
 		Type:   common.REST,
 		Host:   host,

Original file line number	Diff line number	Diff line change
`@@ -69,6 +69,9 @@ type RESTAddress struct {`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`func NewRESTAddress(host string, port int, httpMethod string, scheme string) Address {`
	`72`	`+ if scheme == "" {`
	`73`	`+ scheme = common.HTTP`
	`74`	`+ }`
`72`	`75`	`return Address{`
`73`	`76`	`Type: common.REST,`
`74`	`77`	`Host: host,`