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.

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

Vend tracing instrumentation as a Plugin #1738

Merged
merged 3 commits into from
Sep 26, 2022
Merged

Vend tracing instrumentation as a Plugin #1738

merged 3 commits into from
Sep 26, 2022

Conversation

hlbarber
Copy link
Contributor

@hlbarber hlbarber commented Sep 15, 2022 •
edited
Loading

Motivation and Context

Following #1736 we are able to vend tracing instrumentation as a Plugin.

Description

  • Add Sensitivity trait.
  • Add InstrumentLayer which applies InstrumentOperation.
  • Add Sensitivity implementation in ServerOperationRegistryGenerator
  • Add InstrumentPlugin and InstrumentExt extension trait.

@smithy-lang smithy-lang deleted a comment from github-actions bot Sep 15, 2022
Base automatically changed from external-layers to main September 16, 2022 08:53
@smithy-lang smithy-lang deleted a comment from github-actions bot Sep 16, 2022
@smithy-lang smithy-lang deleted a comment from github-actions bot Sep 16, 2022
@hlbarber hlbarber changed the base branch from main to harryb/update-sensitive September 20, 2022 18:11
@smithy-lang smithy-lang deleted a comment from github-actions bot Sep 20, 2022
@smithy-lang smithy-lang deleted a comment from github-actions bot Sep 20, 2022
@smithy-lang smithy-lang deleted a comment from github-actions bot Sep 20, 2022
@hlbarber hlbarber marked this pull request as ready for review September 20, 2022 18:33
@hlbarber hlbarber requested a review from a team as a code owner September 20, 2022 18:33
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

Base automatically changed from harryb/update-sensitive to main September 26, 2022 15:33
@hlbarber hlbarber removed the blocked label Sep 26, 2022
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@hlbarber hlbarber merged commit 42a8457 into main Sep 26, 2022
@hlbarber hlbarber deleted the harryb/vend-logging branch September 26, 2022 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@82marbag 82marbag 82marbag approved these changes

Assignees
No one assigned
Labels
server Rust server SDK
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hlbarber @82marbag