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

docs: create security assessment #42

Merged

Conversation

szymonkowalczykzf
Copy link
Contributor

@szymonkowalczykzf szymonkowalczykzf commented Feb 20, 2024

Description

Creation of the draft document for Policy Hub Security Assessment

Why

Necessary for the upcoming QG.

Issue

eclipse-tractusx/sig-security#56
eclipse-tractusx/sig-release#504

Checklist

Creation of the draft document for Policy Hub Assessment
Copy link
Contributor

@evegufy evegufy left a comment

Choose a reason for hiding this comment

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

@szymonkowalczykzf Thanks for the draft! I think this diagram (especially the components within the operator subgraph) is bit over-engineered for what we really want to show. Could you please contact me with a meeting to talk about the changes?

De-complicated diagram :)
@szymonkowalczykzf
Copy link
Contributor Author

@evegufy I have probably just commited changes you had in mind.
Still if you would like to have a quick call I am available today 16-17 CET.
Wednasday I am off.
Thursday - 10-13 CET I am available. If one of those slots would work out for you, please confirm which one and I will share invitation.
Cheers!
Szymon

Copy link
Contributor

@evegufy evegufy left a comment

Choose a reason for hiding this comment

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

Thanks, your changes were enough so that I saw the sense in commenting again.

docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
Update includes filling in the missing parts of the assessment:
- Product Description
- Important Links
-Security Checks

Chsanges in the code for product Mermaid Diagram
@evegufy evegufy changed the title Create Security_Assessment.md docs: create security assessment Feb 26, 2024
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
- SPDX-FileCopyrightText: 2021-2023 Contributors to the Eclipse Foundation
- Source URL: https://github.com/eclipse-tractusx/policy-hub


Copy link
Contributor

Choose a reason for hiding this comment

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

please remove extra line breaks

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
docs/technical-documentation/Security_Assessment.md Outdated Show resolved Hide resolved
@evegufy evegufy mentioned this pull request Feb 27, 2024
9 tasks
(Docs) - Updated the security assessment file.
Update contains small, quality related changes.
One small quality related change to the diagram overview.
Changed the version from draft to assessment report
@evegufy evegufy merged commit 716f057 into eclipse-tractusx:release/v0.1.0-rc.4 Feb 27, 2024
7 checks passed
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.

2 participants