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

Fix http attributes of AWS SDK V2 instrumentation #8931

Merged
merged 6 commits into from
Jul 12, 2023
Merged

Fix http attributes of AWS SDK V2 instrumentation #8931

merged 6 commits into from
Jul 12, 2023

Commits on Jul 12, 2023

  1. Fix http attributes of AWS SDK V2 instrumentation 

    Populate http span attributes during beforeTransmission in the
AWS SDK v2 instrumentation so that a request is instrumented
after it is fully transformed by other interceptors.

Signed-off-by: Raphael Silva <rapphil@gmail.com>
    @rapphil
    rapphil committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    580a225 View commit details
    Browse the repository at this point in the history

  2. Update instrumentation/aws-sdk/aws-sdk-2.2/testing/src/main/java/io/o… 

    …pentelemetry/instrumentation/awssdk/v2_2/AbstractQueryProtocolModelTest.java

Co-authored-by: Jean Bisutti <jean.bisutti@gmail.com>
    @rapphil @jeanbisutti
    rapphil and jeanbisutti committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    3e4ca68 View commit details
    Browse the repository at this point in the history

  3. Update instrumentation/aws-sdk/aws-sdk-2.2/javaagent/src/test/java/Qu… 

    …eryProtocolModelTest.java

Co-authored-by: Jean Bisutti <jean.bisutti@gmail.com>
    @rapphil @jeanbisutti
    rapphil and jeanbisutti committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    275f2a1 View commit details
    Browse the repository at this point in the history

  4. Fix: follow conventions in the tests 

    Signed-off-by: Raphael Silva <rapphil@gmail.com>
    @rapphil
    rapphil committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    dd61b7b View commit details
    Browse the repository at this point in the history

  5. Feat: remove dependency on the testLatestDeps property 

    Signed-off-by: Raphael Silva <rapphil@gmail.com>
    @rapphil
    rapphil committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    5a79031 View commit details
    Browse the repository at this point in the history

  6. Improve comments 

    Signed-off-by: Raphael Silva <rapphil@gmail.com>
    @rapphil
    rapphil committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    5f6ce48 View commit details
    Browse the repository at this point in the history