Ruins He

Results 20 comments of Ruins He

Hi, do you have plan to merge this PR into master? Just to confirm, it will fix #2574 right?

我在对这个PR进行测试,好像它没有符合预期,也许是我项目配置版本引用的问题(确认OK了我会做下回复),目前简单验证了一下好像还是会丢参数的样子。

你们好,我在做验证的时候有发现一个问题,但是我不确定我的解决方案是否合理,可以讨论一下,具体的背景如下: ```protobuf service SampleController { rpc Start(StartRequest) returns (StartResponse) { option (google.api.http) = { post: "/api/v1/samples/{definition_key}:start" body: "data" }; } } message Sample { repeated OtherObject sample_value = 1; }...

Confirmed it works when patching this code in `proto_encode.go`: ```diff func encodeValues(msg interface{}, forceTextName bool) (url.Values, error) { if msg == nil || (reflect.ValueOf(msg).Kind() == reflect.Ptr && reflect.ValueOf(msg).IsNil()) { return...

TL;DR using the command line below: ```sh kubectl patch -n kube-system rc registry -p '{"spec":{"template":{"spec":{"containers":[{"name":"registry","image": "registry.cn-hangzhou.aliyuncs.com/google_containers/registry:2.7.1"}]}}}}' kubectl delete pod -n kube-system -l actual-registry=true ``` FYI, just a temporary fix, you...

可以试试最新版,附件的位置设置成 `{{myType}}/附件`,当然可以加自己的前缀(有必要的话)。

Currently I modified the `bashBin` in bash_tool.go in init function directly, if you think it's not a good choice and prefer passing the bash command path using parameters, or using...

> Thanks @ruinshe you will need to rebase the PR because we had to update the presubmits, sorry :( done

Thanks for the detailed roadmap! I had an idea about security, I think we should find ways to avoid passing sensitive key information (such as values of Kubernetes Secrets) between...

Do you use this flag in the latest stable version v0.0.11? I think it was introduced in commit 8a90e07 and has not yet been released.