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

Should build-public delete licensing? #211

Open
mmarseu opened this issue Jun 20, 2024 · 4 comments
Open

Should build-public delete licensing? #211

mmarseu opened this issue Jun 20, 2024 · 4 comments
Milestone

Comments

@mmarseu
Copy link
Collaborator

mmarseu commented Jun 20, 2024

The information in the component.licenses[].license.licensing property is typically only needed internally for license compliance and might even be considered sensitive. After all, nobody should need to know that company A has purchased 3 developer licenses to library X or an enterprise license for framework Y.

What do you think, should build-public delete the licensing field?

@italvi
Copy link
Collaborator

italvi commented Jun 25, 2024

Though you are right that such details should not be exposed to the outside, I think other fields within licensing can also be important for e.g. integrator of a product, i.e. licensing.licenseTypes[] or licensing.expiration, so that I know the potential risk of using a product that does not check whether the components are up-to-date. Your opinion on that? Maybe I am missing something but in my opinion build-public should only delete entries per default that are internal according to the specification, e.g. properties within the namespace internal.

@mmarseu
Copy link
Collaborator Author

mmarseu commented Jun 25, 2024

I follow you logic. It's just that we IMO need some way to delete licensing details (or maybe other information) which is deemed sensitive by the author. Right now, our feature can only delete entire components and properties.
I'm not sure how else to do it. Of course, we could always make a feature like that optional with a command-line switch but maybe there is a more generic way to do it that I haven't thought of?

@italvi italvi added this to the 1.0.0 milestone Jul 3, 2024
@italvi
Copy link
Collaborator

italvi commented Jul 10, 2024

Maybe a config-file or as suggested by you command-line switch, where you can state which fields to delete could be a possible idea for build-public? Though then the question would be: Delete this field everywhere or just for some specific components fulfilling criteria XY?

@mmarseu
Copy link
Collaborator Author

mmarseu commented Jul 10, 2024

How about creating our own property taxonomy for cdx-ev like other's have already done? This could allow us to introduce fine-grained "config" options as properties.
For my particular use-case that would be fine, because the licensing information is added manually in any case. No reason why I couldn't add such a property at the same time to make build-public delete it.

Though that sort of "officialness" (is that a word?) would IMO require us to be a little more stable. Once we go that way, we shouldn't just go and modify properties willy-nilly as we do features of this tool for now.

@italvi italvi modified the milestones: 1.0.0, 1.x.0 Oct 1, 2024
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