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

Add references to TAP 13 (once it is accepted) #198

Closed
pattivacek opened this issue Dec 8, 2020 · 18 comments
Closed

Add references to TAP 13 (once it is accepted) #198

pattivacek opened this issue Dec 8, 2020 · 18 comments
Milestone

Comments

@pattivacek
Copy link
Collaborator

We are removing references to TAP 5, which has been more or less replaced by TAP 13, but TAP 13 isn't approved yet. Once it is, we should restore and rework some of that text.

@pattivacek pattivacek added this to the Uptane 1.2.0 milestone Dec 8, 2020
@jhdalek55
Copy link
Contributor

@trishankatdatadog @mnm678 @JustinCappos what is the current status of TAP 13? Do you think it will be accepted before our June deadline for V.1.2.0? If not, we should change the milestone on this issue.

@JustinCappos
Copy link
Contributor

JustinCappos commented Mar 15, 2021 via email

@plapczyn
Copy link
Contributor

plapczyn commented Apr 27, 2021

Per 2021-04-27 Uptane meeting, we think it should be plausible to release TAP 13 before the 1.2.0 Uptane release. @mnm678 to add to next TUF meeting agenda.

@jhdalek55
Copy link
Contributor

@JustinCappos @mnm678 should we still anticipate this as a 1.2.0 issue?

@jhdalek55
Copy link
Contributor

During the 5/25 Uptane Standards meeting it was decided we would re-label this for V.2.0.0. Though there has been some progress on TAP 13, it likely will require a little more time before approval.

@jhdalek55
Copy link
Contributor

jhdalek55 commented May 25, 2021

I can't change the milestone. Can someone update this for 2.0.0?

@mnm678 mnm678 modified the milestones: Uptane 1.2.0, Uptane 2.0.0 May 25, 2021
@jhdalek55
Copy link
Contributor

Thanks, Marina.

@jhdalek55
Copy link
Contributor

jhdalek55 commented Nov 3, 2021

TAP 13 has been approved as a draft as of 11-3-21. Is this sufficient acceptance to reference it in Uptane? Or, should we wait until it has been approved and merged?
@JustinCappos @tkfu @trishankatdatadog @iramcdonald...what are your thoughts?

@mnm678
Copy link
Collaborator

mnm678 commented Nov 3, 2021

We should wait until it's approved. Some of the implementation details might change, so Uptane implementers should probably wait to implement until acceptance.

@jhdalek55
Copy link
Contributor

Thanks @mnm678. That makes sense.

@jhdalek55
Copy link
Contributor

Since this issue cannot be resolved at this time, can we make a new label, perhaps PENDING, for issues like this for which cannot say when they answered? I would prefer we did not leave issues like this (#162 #198) labeled "2.0.0" on the repo as it gives the appearance that we did not completed these on-time. Failing that can we leave it posted without a version label at all?

@jhdalek55
Copy link
Contributor

This is still waiting for TUF approval of TAP 13.

@jhdalek55
Copy link
Contributor

I'm pasting in the original references to TAP 5 that were removed. In all these cases, the reference to TAP was taken out, but nothing was substituted. Given that TAP 13 has probably changed a bit over the past three years, would it still be a relevant reference? If not, do we need to bother with this at all, or could we just close this issue?

5.2.2. Root metadata: "If this mapping of URLs is used, the implementer SHOULD implement this functionality following [TAP-5] to avoid adding unforeseen security risks."

5.2.4. Snapshot metadata: "The Snapshot metadata MAY also list the Root metadata filename and version number. This is not required, particularly for implementations of [TAP-5], but MAY be included in all cases for backward compatibility."

5.4.4.2. Full verification: "If [TAP-5] is supported and a Primary has an external connection to the Uptane repositories, a Primary ECU SHALL download metadata and images following the rules specified in that TAP."

@JustinCappos
Copy link
Contributor

@mnm678 Is the right person to look at this, I think.

@mnm678
Copy link
Collaborator

mnm678 commented Nov 22, 2022

TAP 13 only allows mapping for targets files, so isn't relevant in the 5.2.4 case at all.

In general, I think we could just close this until someone requests this feature specifically for Uptane. TAP 13 will work with Uptane, but I think we can leave it out of the standard until there's a need for it.

@jhdalek55
Copy link
Contributor

It mnkes sense to me as well to close it, at least for now. Let's leave leave the PR open maybe for a week or so for any dissenting opinions, and then this we'll close it.

@jhdalek55
Copy link
Contributor

@mnm678 can you close this, per the decision at the 11/22 Standards meeting? There have been no objections over the past 10 days so I think we're good to go.

@mnm678 mnm678 closed this as completed Dec 2, 2022
@jhdalek55
Copy link
Contributor

Thanks, @mnm678 .

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

5 participants