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

Mac 无法运行,window/linux 下是正常的,麻烦大佬,看一下 #58

Closed
sqeven opened this issue Jun 8, 2022 · 5 comments
Closed

Comments

@sqeven
Copy link

sqeven commented Jun 8, 2022

github.com/emmansun/gmsm/smx509

../../../../pkg/mod/github.com/emmansun/gmsm@v0.12.2/smx509/cert_pool.go:112:9: undefined: loadSystemRoots
../../../../pkg/mod/github.com/emmansun/gmsm@v0.12.2/smx509/root.go:17:32: undefined: loadSystemRoots
../../../../pkg/mod/github.com/emmansun/gmsm@v0.12.2/smx509/verify.go:648:4: not enough arguments to return
../../../../pkg/mod/github.com/emmansun/gmsm@v0.12.2/smx509/verify.go:648:12: c.systemVerify undefined (type *Certificate has no field or method systemVerify)
../../../../pkg/mod/github.com/emmansun/gmsm@v0.12.2/smx509/verify.go:651:28: c.systemVerify undefined (type *Certificate has no field or method systemVerify)

@zhangyongding
Copy link

一样的问题

@emmansun
Copy link
Owner

emmansun commented Jun 9, 2022

当时看到darwin依赖很多golang sdk internal package,所以就忽略了。我现在推了一个临时方案:darwin不考虑操作系统自己的证书链。这个改动会发布到v0.12.3.

@sqeven
Copy link
Author

sqeven commented Jun 9, 2022

好的,谢谢

@sqeven sqeven closed this as completed Jun 9, 2022
@xuyang2
Copy link
Contributor

xuyang2 commented Jun 9, 2022

@emmansun
func (c *Certificate) Verify(opts VerifyOptions)
中的 if runtime.GOOS == "windows"
是否需要改成 if runtime.GOOS == "windows" || runtime.GOOS == "darwin"

if runtime.GOOS == "windows" {

@emmansun
Copy link
Owner

emmansun commented Jun 9, 2022

@emmansun func (c *Certificate) Verify(opts VerifyOptions) 中的 if runtime.GOOS == "windows" 是否需要改成 if runtime.GOOS == "windows" || runtime.GOOS == "darwin"

if runtime.GOOS == "windows" {

现在不需要,目前的root_darwin.go 没有真正实现systemVerify()方法。

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

No branches or pull requests

4 participants