We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The demands and suggestions of IOC-golang in practical projects. Including API, concepts, architecture and all related issue can be discussed here.
IOC-golang 在实际工程中的诉求和建议。 包括API、概念、架构相关的内容都可以在这里讨论。
The text was updated successfully, but these errors were encountered:
IOC-golang/aop/proxy.go
Lines 115 to 118 in 6fb33a0
这里的实现,如果在before中有panic的话会导致after执行不到,而且panic会被淹没, 请问下这里是怎么考虑的? 是否考虑改为将after放在defer中来确保一定会被执行到,并且有panic的话再继续把panic传递下去?
Sorry, something went wrong.
IOC-golang/aop/proxy.go Lines 115 to 118 in 6fb33a0 for _, i := range interceptorImpls { i.AfterInvoke(invocationCtx) } 这里的实现,如果在before中有panic的话会导致after执行不到,而且panic会被淹没, 请问下这里是怎么考虑的? 是否考虑改为将after放在defer中来确保一定会被执行到,并且有panic的话再继续把panic传递下去?
已提pr解决,具体见该pr:#115
No branches or pull requests
The demands and suggestions of IOC-golang in practical projects. Including API, concepts, architecture and all related issue can be discussed here.
IOC-golang 在实际工程中的诉求和建议。
包括API、概念、架构相关的内容都可以在这里讨论。
The text was updated successfully, but these errors were encountered: