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

Influxdb 1.8.3 checksum changed again! #391

Open
karthick-rn opened this issue May 5, 2021 · 4 comments
Open

Influxdb 1.8.3 checksum changed again! #391

karthick-rn opened this issue May 5, 2021 · 4 comments

Comments

@karthick-rn
Copy link
Contributor

This is the same problem we faced in Dec 2020 as well and discussed here in #381. Looks like someone already opened an issue - influxdata/influxdb#21365.

@karthick-rn
Copy link
Contributor Author

I'm not clear why they had to re-generate all signature files and rotate GPG keys, but looks like it is their process. I'll submit a PR to update the new checksum unless anyone has any thoughts.

@ctubbsii
Copy link
Member

ctubbsii commented May 6, 2021

Every time the checksum changes, it is suspicious. I don't think we should just keep blindly updating it going forward, because that would be like it didn't have a checksum at all. We could manually check every time, but that's tedious and requires a copy of both the old and new artifact (which may not be possible every time this happens).

So, I think the best solution is to try to convince upstream that their process is flawed, that it creates confusion and sows distrust in their security. If we can't rely on the checksum not changing for a previously released version, that's pretty concerning.

In my opinion, the second best solution is to remove features from muchos that use InfluxDB. If we can't trust the dependency, we should avoid it.

The third best solution seems to manually check that only the signature changed (as I did in #381 (comment)). But, that may not be possible.

@karthick-rn
Copy link
Contributor Author

Added a comment to the Influxdb issue.

@karthick-rn
Copy link
Contributor Author

The existing RPMs has been re-signed again today!

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants