Skip to content
This repository was archived by the owner on Feb 14, 2024. It is now read-only.

combine updater and tuf packages #22

Open
groob opened this issue Jun 27, 2017 · 0 comments
Open

combine updater and tuf packages #22

groob opened this issue Jun 27, 2017 · 0 comments
Assignees
@groob

Comments

@groob
Copy link
Contributor

groob commented Jun 27, 2017

based on slack discussion:

If we merge #20, we're exposing the tuf client to the outside world, which increases the complexity of the implementation and state management somewhat.

We should merge the updater package into tuf, making the client private but allowing the caller to:

a) continuously monitor a specific target for changes
b) download a specific target based on current local repo metadata
@groob groob self-assigned this Jun 27, 2017
@groob groob mentioned this issue Jun 27, 2017
Merged
groob added a commit that referenced this issue Jun 27, 2017
@groob
add download method for updater (#20)
04e45b3 
* add download method for updater

Closes #13

In the process of adding the method also discovered and updated the potentially race-y usage of the repoManager struct.

Closes #16 but full fix should come in with #22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@groob groob

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@groob