Node.js layer v0.24.0 does not work with @opentelemetry/winston-transport v0.35.0

Đang mở
  • Có gì không ổn?
#2,723 2 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 2 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
57/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
aws, go, javascript, node.js
Lĩnh vực
backend-api-design, observability

Hướng nghiên cứu

Start by reading the Node.js layer's log export and instrumentation setup, then compare its OpenTelemetry log API dependencies with @opentelemetry/winston-transport 0.35.0; the issue names no source files or tests. Reproduce with the listed Lambda, Winston, and transport versions, then confirm logs export or that a clear warning appears for an unsupported version.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug javascript

Describe the bug
We're using the nodejs-layer 0.24.0 and the @opentelemetry/winston-transport for logging.
With the recently released @opentelemetry/winston-transport 0.35.0 however, logs are no longer exported by the nodejs-layer. Previously, with @opentelemetry/winston-transport 0.34.0, this was working great.

I suppose this is with the recent switch in @opentelemetry/winston-transport 0.35.0 to use @opentelemetry/api 1.10.0. In 0.34.0, it used the dependency @opentelemetry/api-logs 0.223.0. (Note "api-logs" instead of "api")

Steps to reproduce

  1. Create an AWS Node.js 24 Lambda with the nodejs-layer 0.24.0.
  2. Install winston 3.x and @opentelemetry/winston-transport 0.35.0.
  3. Deploy some code which creates a default winston logger and produces some logs.

What did you expect to see?
I expect the nodejs-layer to export logs as it used to do.

If that is not possible because of a version mismatch, I expect to see some kind of warning/error log in CloudWatch, instead of it being silently broken.

What did you see instead?
I did not see logs being exported, neither did I get any warning/error.

Additional context
Add any other context about the problem here.

Tip: React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.

Ngôn ngữ chính
Go
Star
433
Fork
249
Merge trung bình
1 ngày 12 giờ
Pull request đã merge (30 ngày)
39

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của open-telemetry/opentelemetry-lambda

Tất cả issue của open-telemetry/opentelemetry-lambda

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.