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

Remove references to TUF #75

Closed
vladimir-v-diaz opened this issue Nov 2, 2017 · 2 comments
Closed

Remove references to TUF #75

vladimir-v-diaz opened this issue Nov 2, 2017 · 2 comments

Comments

@vladimir-v-diaz
Copy link
Contributor

Description of issue or feature request:

securesystemslib was not always a library. Many of its modules and functions previously lived in the TUF repository. Unfortunately, there are still references to TUF in some of the securesystemslib comments.

Current behavior:
There are references to TUF in comments and there exist schemata that are only relevant to TUF. For example:

A central location for all format-related checking of TUF objects.
Note: 'formats.py' depends heavily on 'schema.py', so the 'schema.py'
module should be read and understood before tackling this module.
'formats.py' can be broken down into three sections. (1) Schemas and object
matching. (2) Classes that represent Role Metadata and help produce
correctly formatted files. (3) Functions that help produce or verify TUF

# Version information specified in "snapshot.json" for each role available on
# the TUF repository. The 'FILEINFO_SCHEMA' object was previously listed in
# the snapshot role, but was switched to this object format to reduce the
# amount of metadata that needs to be downloaded. Listing version numbers in
# "snapshot.json" also prevents rollback attacks for roles that clients have
# not downloaded.
VERSIONINFO_SCHEMA = SCHEMA.Object(
object_name = 'VERSIONINFO_SCHEMA',
version = METADATAVERSION_SCHEMA)

Expected behavior:
TUF references in securesystemslib should be removed and TUF-related schemata relocated to the TUF repository.

@JustinCappos
Copy link

This is a low priority, cleanup item...

@lukpueh
Copy link
Member

lukpueh commented Sep 6, 2019

Fixed with #165.

@lukpueh lukpueh closed this as completed Sep 6, 2019
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

3 participants