You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@dubbo.apache.org by "KamToHung (via GitHub)" <gi...@apache.org> on 2025/08/16 15:06:52 UTC
[PR] Update the dependencies version and fix compatibility/api used error [dubbo-go-samples]
KamToHung opened a new pull request, #892:
URL: https://github.com/apache/dubbo-go-samples/pull/892
update dubbo-go, gost version and fix compatibility/api used error
Compatibility/proxyless/go - the client subdirectory of the go. The mod file attempts to import github.com/apache/dubbo-go-samples/compatibility/api package, However, this package does not exist in the remote repository because it is a local module.
<img width="1760" height="766" alt="image" src="https://github.com/user-attachments/assets/5ae3b308-230d-480b-ada9-48f89d45ae40" />
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org
Re: [PR] Update the dependencies version and fix compatibility/api used error [dubbo-go-samples]
Posted by "AlexStocks (via GitHub)" <gi...@apache.org>.
AlexStocks merged PR #892:
URL: https://github.com/apache/dubbo-go-samples/pull/892
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org
Re: [PR] Update the dependencies version and fix compatibility/api used error [dubbo-go-samples]
Posted by "AlexStocks (via GitHub)" <gi...@apache.org>.
AlexStocks commented on code in PR #892:
URL: https://github.com/apache/dubbo-go-samples/pull/892#discussion_r2280467667
##########
compatibility/proxyless/go-client/cmd/client.go:
##########
@@ -19,6 +19,8 @@ package main
import (
"context"
+
+ "github.com/apache/dubbo-go-samples-mesh-client/api"
Review Comment:
这里位置放错了
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org
Re: [PR] Update the dependencies version and fix compatibility/api used error [dubbo-go-samples]
Posted by "Copilot (via GitHub)" <gi...@apache.org>.
Copilot commented on code in PR #892:
URL: https://github.com/apache/dubbo-go-samples/pull/892#discussion_r2280465405
##########
compatibility/proxyless/go-server/api/generate.sh:
##########
@@ -0,0 +1,4 @@
+export GO111MODULE="on"
+export GOPROXY="http://goproxy.io"
Review Comment:
Using HTTP for GOPROXY instead of HTTPS could expose dependencies to man-in-the-middle attacks. Consider using 'https://goproxy.io' or 'https://proxy.golang.org' instead.
```suggestion
export GOPROXY="https://goproxy.io"
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org
Re: [PR] Update the dependencies version and fix compatibility/api used error [dubbo-go-samples]
Posted by "AlexStocks (via GitHub)" <gi...@apache.org>.
AlexStocks commented on code in PR #892:
URL: https://github.com/apache/dubbo-go-samples/pull/892#discussion_r2280467773
##########
compatibility/proxyless/go-server/cmd/server.go:
##########
@@ -19,6 +19,8 @@ package main
import (
"context"
+
+ "github.com/apache/dubbo-go-samples-mesh-server/api"
Review Comment:
位置放错了，改回去吧
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org