Skip to content

Add BSI-TR-03185-2 reference IDs to OSPS-LE.yaml#465

Open
SecurityCRob wants to merge 2 commits intomainfrom
SecurityCRob-patch-14
Open

Add BSI-TR-03185-2 reference IDs to OSPS-LE.yaml#465
SecurityCRob wants to merge 2 commits intomainfrom
SecurityCRob-patch-14

Conversation

@SecurityCRob
Copy link
Contributor

legal things for bsi mappings

legal things for bsi mappings

Signed-off-by: CRob <69357996+SecurityCRob@users.noreply.github.com>
@SecurityCRob
Copy link
Contributor Author

SecurityCRob commented Jan 13, 2026

depends on:
#459
#460

related to:
#461
#462
#463
#464
#465
#466
#467

entries:
- reference-id: LE.01
assessment-requirements:
- id: OSPS-LE-02.01

Choose a reason for hiding this comment

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

OSPS-LE-03.02 seems more aligned in this case. @eddie-knight

" OSPS-LE-03.02: While active, the license for the released software assets MUST be included in the released source code, or in a LICENSE file, COPYING file, or LICENSE/ directory alongside the corresponding release assets. "

Copy link
Contributor Author

Choose a reason for hiding this comment

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

LE.01 is part of the original BSI mappings and I would encourage us not to adjust without their input. We certainly can add LE-03 as part of our assessment for the mapping as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

Bit confused; same comment as the other PR: #464 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct, the same comment was made for both prs. I took whatever the BSI wrote for mappings and added items I felt were missing. I do not desire to remove what they wrote as the basis. If the team feels strongly about it they could reach out to the BSI and attempt to persuade them to adjust the source material that then later can be reflected in OSPS.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we trying to replicate their mappings as our own, instead of creating our own outgoing mappings with our own claims regarding how well our entries address theirs?

My comment on both PRs is that we should treat these mappings as our own statements.

Copy link
Contributor

@evankanderson evankanderson Jan 30, 2026

Choose a reason for hiding this comment

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

The BSI mapping looks correct.

OSPS-LE-02 says that the licenses must (be stated and) meet the OSI/FSF definitions.

OSPS-LE-03 says that the licenses must be distributed in standard locations (matching LE.02's requirement for a copy of the license to be distributed).

As an example, a repo which has MIT header on every file but doesn't include a copy of the MIT license would technically pass OSPS-LE-02/BSI LE.01 (though it might trip up automation), but would fail OSPS-LE-03 / BSI LE.02.

entries:
- reference-id: LE.01
assessment-requirements:
- id: OSPS-LE-02.01
Copy link
Contributor

@evankanderson evankanderson Jan 30, 2026

Choose a reason for hiding this comment

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

The BSI mapping looks correct.

OSPS-LE-02 says that the licenses must (be stated and) meet the OSI/FSF definitions.

OSPS-LE-03 says that the licenses must be distributed in standard locations (matching LE.02's requirement for a copy of the license to be distributed).

As an example, a repo which has MIT header on every file but doesn't include a copy of the MIT license would technically pass OSPS-LE-02/BSI LE.01 (though it might trip up automation), but would fail OSPS-LE-03 / BSI LE.02.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants