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.

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

Outdated OTEL dependencies prevent import #1148

Open
4 of 5 tasks
nico151999 opened this issue Jan 31, 2024 · 0 comments
Open
4 of 5 tasks

Outdated OTEL dependencies prevent import #1148

nico151999 opened this issue Jan 31, 2024 · 0 comments
Labels
bug Something is not working.

Comments

@nico151999
Copy link

nico151999 commented Jan 31, 2024

Preflight checklist

Ory Network Project

No response

Describe the bug

The open telemetry libraries have changed their folder structure over time and consequently the packages one needs to import. Oathkeeper offers a gRPC middleware that one can only use if ones does not depend on newer open telemetry packages with a different folder structure. The outdated dependencies keep one from importing oathkeeper in such projects. I thought about implementing a connect-go middleware based on the gRPC implementation so that I can use oathkeeper in my private connect-go services directly without downgrading my OTEL dependencies or hosting a dedicated instance of oathkeeper.

Reproducing the bug

Start a new Golang module and import current versions of open telemetry. Then, import oathkeeper, too. You will get an error due to the different folder structure of the packages in the different OTEL module versions.

Relevant log output

No response

Relevant configuration

No response

Version

v0.40.6

On which operating system are you observing this issue?

Linux

In which environment are you deploying?

Kubernetes with Helm

Additional Context

My connect-go microservices are deployed to a Kubernetes cluster but the oathkeeper should become part of them. As I intend not to use a dedicated Oathkeeper instance I am currently not using the Oathkeeper Helm chart but I would use it as a fallback if you do not consider updating the OTEL dependencies. UPDATE: this fallback is not an option since a dedicated Oathkeeper instance cannot handle gRPC traffic.

@nico151999 nico151999 added the bug Something is not working. label Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working.
Projects
None yet
Development

No branches or pull requests

1 participant