Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go-client 泛化调用go-server #801

Closed
yinitian opened this issue Oct 22, 2020 · 2 comments · Fixed by #822
Closed

go-client 泛化调用go-server #801

yinitian opened this issue Oct 22, 2020 · 2 comments · Fixed by #822
Assignees
Milestone

Comments

@yinitian
Copy link

Time.Time类型在传输过程中变成了nil,无论是是从client请求到server还是从server返回到client

另外自定义了重命名 Time.Time MyTime,增加序列化,情况依然没有变化

2020/10/22 21:22:24 dubbo client: ManageProvider Login [{admin 123456}]
2020/10/22 21:22:24 dubbo resp: map[createdAt: deletedAt: email:[email protected] id:1 password:250415709c2a1e522232f86a7dbe553b salt:5Id0Ag state:1 token:eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6MSwidXNlcl9uYW1lIjoiYWRtaW4iLCJlbWFpbCI6Inlhbmd4aWFvbGVpQGt1YWlmYXpzLmNvbSIsImJ1c2luZXNzIjozLCJleHAiOjE2MDMzODM3NDQsImlzcyI6ImFkc2Vuc2UtbWFuYWdlciJ9.3VG6LIENfPIogg5a9Fzy5fB3Ag0w_RXfGC0LCGiHfu4 updatedAt: userName:admin]

[2020-10-22/21:51:25 kuaifa.com/adsense/adsense/server/advertiser.(*AdvertiserProvider).Register: advertiser.go: 20] req:advertiser.Advertiser{Id:0, ScreenName:"杀猪的", Mobile:"18600363056", Qq:"12232", Wechat:"1111", AccountType:1, Balance:0, CertificatesImg:"11", BusinessScope:"", Status:0, Adjust:2, Sfrom:1, CreatedAt:time.Time{wall:0x0, ext:0, loc:(*time.Location)(nil)}, UpdatedAt:time.Time{wall:0x0, ext:0, loc:(*time.Location)(nil)}, Token:""}

另外咨询个问题,泛化调用的时候能不能声明返回值类型

@fangyincheng
Copy link
Contributor

泛化调用就是为了去除客户端的dto定义,为啥要声明返回值类型?

@zouyx zouyx linked a pull request Oct 30, 2020 that will close this issue
@zouyx zouyx added this to the v1.5.4 milestone Oct 30, 2020
@zouyx
Copy link
Member

zouyx commented Nov 2, 2020

@yinitian you can use dubbo-go v1.5.4 for test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants