bitmaelum / bitmaelum-suite

BitMaelum software suite
https://bitmaelum.com
MIT License
61 stars 5 forks source link

See if we can get rid of the scary opentelemetry package #125

Closed jaytaph closed 3 years ago

jaytaph commented 3 years ago

Some package we use, depends on opentelemetry.. this sounds scary (even though harmless, as we don't do any telemetry whatsoever).

Just to make sure we don't get strange discussions, see if we can remove it somehow..

jaytaph commented 3 years ago

Unfortunately, we cannot. The go-redis/redis package uses this for adding telemetry to redis.

acalatrava commented 3 years ago

I think we used an old version of the reds package so we can get rid of this. Is this no longer possible?

jaytaph commented 3 years ago

i think we are on 8.4.4, which is the latest.. it still got the opentelemetry reference in it :(

jaytaph commented 3 years ago

Downgraded to v7, which does not have telemetry packages.