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.

Sign up for GitHub

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

Jump to bottom

fix: don't copy message to OnRequestHook #1012

Merged

guillaumemichel merged 1 commit into master from fix/onRequestHook

Jan 28, 2025

Contributor

guillaumemichel commented Jan 23, 2025 •

edited

Loading

Fix to #1011

go vet isn't happy about copying pb.Message since it contains a lock in #975.


          fix: don't copy message to OnRequestHook

44e76f5

guillaumemichel requested a review from dennis-tra

January 23, 2025 14:36

guillaumemichel requested a review from a team as a code owner

January 23, 2025 14:36

guillaumemichel merged commit 811669a into master

9 checks passed

guillaumemichel deleted the fix/onRequestHook branch

January 28, 2025 12:55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet