Skip to content
This repository was archived by the owner on Oct 31, 2024. It is now read-only.

Commit ee0f38d

Browse files
author
Amir Blum
authored
Merge pull request #7 from aspecto-io/plugin-aws-cleanup
cleanup in plugin-aws-sdk
2 parents 7bd68a4 + 0aefe80 commit ee0f38d

File tree

3 files changed

+4
-2
lines changed

3 files changed

+4
-2
lines changed

packages/plugin-aws-sdk/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@
88
"aws-sdk"
99
],
1010
"author": "Victor Livar <[email protected]>",
11+
"contributors": [
12+
"Amir Blum <[email protected]>"
13+
],
1114
"homepage": "https://github.com/aspecto-io/opentelemetry-ext-js",
1215
"license": "Apache-2.0",
1316
"main": "dist/src/index.js",

packages/plugin-aws-sdk/src/aws-sdk.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,6 @@ class AwsPlugin extends BasePlugin<typeof AWS> {
7676

7777
span = thisPlugin._tracer.startSpan(thisPlugin._getSpanName(this), {
7878
attributes: {
79-
// [AspectoAttributeNames.PLUGIN_NAME]: awsModuleName,
8079
[AttributeNames.COMPONENT]: thisPlugin.moduleName,
8180
[AttributeNames.AWS_OPERATION]: this.operation,
8281
[AttributeNames.AWS_PARAMS]: JSON.stringify(this.params),

packages/plugin-aws-sdk/test/aws-sdk.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import { ContextManager } from "@opentelemetry/context-base";
1212
import { AsyncHooksContextManager } from "@opentelemetry/context-async-hooks";
1313
import { AttributeNames } from "../src/enums";
1414

15-
describe("aspecto-open-telemetry-aws", () => {
15+
describe("plugin-aws-sdk", () => {
1616
const logger = new NoopLogger();
1717
const provider = new NodeTracerProvider();
1818
const memoryExporter = new InMemorySpanExporter();

0 commit comments

Comments
 (0)