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

more data cleanup #2399

Merged
merged 1 commit into from
Sep 9, 2022
Merged

more data cleanup #2399

merged 1 commit into from
Sep 9, 2022

Conversation

kurtseifried
Copy link
Contributor

No description provided.

{
"package": {
"name": "Secure Remote Access (SRA)",
"ecosystem": "GSD"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, isn't the ecosystem supposed to be the ecosystem of the package? e.g., NPM or RubyGems.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There isn't an ecosystem for general software so we use "GSD" https://ossf.github.io/osv-schema/#affectedpackage-field

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an example of the limitations of ecosystem / package, relevant to:

ossf/osv-schema#83

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One option:

{
  "software": {
    "name": "Secure Remote Access (SRA)"
  }
}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's not compliant with the OSV schema

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neither is using an ecosystem of GSD.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oliver had a suggestion for what to temporarily use for unsupported ecosystems somewhere, let me see if I can find it again.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Although also not officially OSV. Has someone already started a conversation on the OSV spec either proposing new ecosystem values or having some way of allowing unsupported ones until there is an official one?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"ecosystem": "GSD"
},
"versions": [
"8.x, 9.0.0.9-26sv and earlier"
Copy link
Contributor

@raphaelahrens raphaelahrens Sep 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be multiple strings ["8.x", "9.0.0.9-26sv and earlier"] as defined here. Also the "and ealier" part is very vague.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again this is part of the problem, bad data, and I'm not going to completely cvlean it up one by one, I'm currently getting the files into json compliant state, a data compliant state is step #4.

Copy link
Member

@joshbuker joshbuker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't agree with using an ecosystem of GSD, but we can fix this later.

@kurtseifried
Copy link
Contributor Author

Exactly. GSD is a horrible value to use. But there's no really better alternative. "software"? "C"? "github" (if hosted there), "OpenSource"? This is about starting the conversation of how we fix and improve the OSV schema and data because ecosystem is well meaning, and works well for some cases (where there is a clear ecosystem), but the world of software/hwardware/services/etc is wide and diverse and ecosystem won't work for all of it. For example:

ossf/osv-schema#83

"We just need clearly defined rules for each ecosystem. There must be no ambiguity as to what a "name" means in an ecosystem. This is not always obvious: e.g. for Debian, this must be source packages, not binary packages. For Python, the package name must be normalized. We can't just have e.g. ecosystem: "", name: "human readable text" as these are not very actionable."

Which... I'm not sure that will work at a meta level at all for us. E.g. Where do debian binary packages go for ecosystem?

As you can see this is problematic in many ways.

@kurtseifried kurtseifried merged commit 79ede89 into CloudSecurityAlliance:main Sep 9, 2022
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

Successfully merging this pull request may close these issues.

5 participants