-
Notifications
You must be signed in to change notification settings - Fork 29
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.
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
Remove pin on opentelemetry-instrumentation-logging
#97
Comments
Got it. Thanks. Curious about the reasoning behind keeping the versions pinned when OpenTelemetry follows semantic versioning. It does seem that'd be stable. |
Hi @setu4993 Thanks for bringing this up! We keep the version pinned mainly for stability and to avoid any surprises with automatic updates. It helps us ensure everything works smoothly and avoids any unexpected issues. Pinning also helps us manage security risks better. We can thoroughly test updates before releasing them, ensuring no new security vulnerabilities are introduced. Even though OpenTelemetry uses semantic versioning, pinning lets us control any potential conflicts and maintain a consistent experience. Hope this helps! Let us know if you have more questions. |
Thanks for that context and engaging on this. From the source I can see I don't think it's particularly risky, but I admit I don't have the complete picture. I hope the Logz.io team will consider removing the pin in the future. Regardless, given this is now updated to atleast the current latest version, I'm going to close this issue. Thanks, again, for jumping on this quickly and explaining your reasoning. |
The package
opentelemetry-instrumentation-logging
is pinned to v0.39.0 which forces the use ofopentelemetry-instrumentation
packages with v0.39b. The instrumentation packages have a CVE that affects 0.39 (resolved in v0.41b): GHSA-5rv5-6h4r-h22vCan the pin please be removed and the package updated to a more recent version?
The text was updated successfully, but these errors were encountered: