Skip to content

Commit

Permalink
Remove otel HTTP_METHOD attributes
Browse files Browse the repository at this point in the history
  • Loading branch information
lkostrowski committed Jan 5, 2024
1 parent e1fd515 commit 5a613ed
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
5 changes: 5 additions & 0 deletions .changeset/chatty-cherries-begin.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@saleor/app-sdk": patch
---

Removed HTTP_METHOD OTEL attributes. They broke displaying of resources when other attributes related to HTTP are not set.
3 changes: 0 additions & 3 deletions src/APL/vercel-kv/vercel-kv-apl.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ export class VercelKvApl implements APL {
attributes: {
saleorApiUrl,
[SemanticAttributes.PEER_SERVICE]: OTEL_APL_SERVICE_NAME,
[SemanticAttributes.HTTP_METHOD]: "GET",
},
kind: SpanKind.CLIENT,
},
Expand Down Expand Up @@ -91,7 +90,6 @@ export class VercelKvApl implements APL {
saleorApiUrl: authData.saleorApiUrl,
appId: authData.appId,
[SemanticAttributes.PEER_SERVICE]: OTEL_APL_SERVICE_NAME,
[SemanticAttributes.HTTP_METHOD]: "POST",
},
kind: SpanKind.CLIENT,
},
Expand Down Expand Up @@ -133,7 +131,6 @@ export class VercelKvApl implements APL {
attributes: {
saleorApiUrl,
[SemanticAttributes.PEER_SERVICE]: OTEL_APL_SERVICE_NAME,
[SemanticAttributes.HTTP_METHOD]: "DELETE",
},
kind: SpanKind.CLIENT,
},
Expand Down

0 comments on commit 5a613ed

Please sign in to comment.