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

Expired metadata for succinct roles #58

Closed
rdimitrov opened this issue Aug 18, 2023 · 8 comments
Closed

Expired metadata for succinct roles #58

rdimitrov opened this issue Aug 18, 2023 · 8 comments

Comments

@rdimitrov
Copy link
Collaborator

I've been using this repo as part of the examples in rdimitrov/go-tuf-metadata and noticed that the metadata for the succinct roles needs to be regenerated as it had expired.

@rdimitrov rdimitrov changed the title Expired succinct roles Expired metadata for succinct roles Aug 18, 2023
@rdimitrov
Copy link
Collaborator Author

@jku
Copy link
Owner

jku commented Aug 21, 2023

I'm trying to decide what to do with this... I'd like to have one or two repositories that are good examples and have actual content changes but I'm not sure if I want to maintain this one as it is.

  • this is built / maintained with repository-editor-for-tuf which is a fine tool for creating whatever metadata you want... but was not intended to be a repository maintenance tool
  • tuf-on-ci is a repository maintenance system and we could switch this to be a tuf-on-ci maintained repo... but that tool does not, at least at this point, support all the things demonstrated in this repo (like succinct delegations)

Considering I don't know what to do, maybe the right path would be to just make the metadata be valid again...

  • find the signing keys, I'm sure they're stored on some disk somewhere
  • create new versions of metadata with 20yr expiry

That said, I do expect the online scripts to break at some point so in some ways this is band aid

@rdimitrov
Copy link
Collaborator Author

I agree 👍 If there's a way to automate the generation of succinct delegations as to how I believe the normal delegations are automated here it would suffice for the time being. Nevertheless, I was happy to see that it handled the expired metadata correctly and it failed.

On the other note, do you plan to add succinct roles to tuf-on-ci?

Now that I think about it - perhaps the way I use tuf-demo in go-tuf-metadata should be just another test case for the conformance testing rig.

@jku
Copy link
Owner

jku commented Aug 21, 2023

On the other note, do you plan to add succinct roles to tuf-on-ci?

I'm not opposed to it, but

  • meaningful succinct delegations likely means online signed targets (because it means a large amount of artifacts and frequent signing) -- so it might be a larger change with significant user experience effect
  • as long as no actual user is asking for this, it's likely not very high on priority list

@jku
Copy link
Owner

jku commented Aug 23, 2023

I think I will turn this into a TUF-on-CI maintained repository: I am going to need a long term example repository for that anyway and I don't want to maintain two. So that will mean the succinct delegations are going to go away.

@rdimitrov
Copy link
Collaborator Author

That makes sense 👍 I will update my use cases accordingly once you make that change.

Nevertheless, I think having a working example repo with succinct delegations has its use cases, but I agree it's not worth achieving it through tuf-on-ci, at least not until there's a better reason 👍

@jku
Copy link
Owner

jku commented Aug 23, 2023

Nevertheless, I think having a working example repo with succinct delegations has its use cases

Yes, if/when we have a conformance test suite, this should be included

@jku
Copy link
Owner

jku commented Oct 4, 2023

Closing as I just removed all metadata (this repository starts from scratch soon)

@jku jku closed this as completed Oct 4, 2023
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