Skip to content

Conversation

@cratelyn
Copy link
Member

@cratelyn cratelyn commented Mar 4, 2025

TapEventExt provides an extension trait interface that we use to extends linkerd_proxy_api::tap::TapEvent with additional interfaces for use in integration tests.

this commit removes request_init_path(). this method was originally added in 3ac6b72 (#154), but was never actually implemented and will only ever panic when invoked. thus, it can be removed.

`TapEventExt` provides an extension trait interface that we use to
extends `linkerd_proxy_api::tap::TapEvent` with additional interfaces
for use in integration tests.

this commit removes `request_init_path()`. this method was originally
added in 3ac6b72 (#154), but was never actually implemented and will
only ever panic when invoked. thus, it can be removed.

Signed-off-by: katelyn martin <[email protected]>
@cratelyn cratelyn marked this pull request as ready for review March 4, 2025 16:02
@cratelyn cratelyn requested a review from a team as a code owner March 4, 2025 16:02
@cratelyn cratelyn enabled auto-merge (squash) March 4, 2025 16:02
@cratelyn cratelyn merged commit 16614ca into main Mar 4, 2025
15 checks passed
@cratelyn cratelyn deleted the kate/app-integration-tap-event-ext-remove-unused-method branch March 4, 2025 16:03
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 this pull request may close these issues.

3 participants