-
Notifications
You must be signed in to change notification settings - Fork 36
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
Rework request handling in memory handler #1443
Rework request handling in memory handler #1443
Conversation
Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1443 +/- ##
=======================================
Coverage ? 70.30%
=======================================
Files ? 242
Lines ? 10999
Branches ? 0
=======================================
Hits ? 7733
Misses ? 2767
Partials ? 499 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
m := new(dns.Msg) | ||
_ = rw.WriteMsg(m.SetRcode(msg, dns.RcodeSuccess)) | ||
} else { | ||
rwWrapper := &responseWriter{ResponseWriter: rw} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- please remove
responseWriter.
fanout
should handler the server fail.
- add regression unit test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
responseWriter
doesn't check only fanout, but any handler that goes after memory handler. If we remove responseWriter
we can get an error, because in some cases nothing will be written to rw
.
Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@glazychev-art Do you have any comments?
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
…k@main PR link: networkservicemesh/sdk#1443 Commit: 2502d93 Author: Nikita Skrynnik Date: 2023-04-10 14:27:13 +0700 Message: - Rework request handling in memory handler (#1443) * rework request handling in memory handler Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * cleanup Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * add test for proper handling of nonexistent records check Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> --------- Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
Description
Issue link
#1425
How Has This Been Tested?
Types of changes