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

Create security_assessment.md #77

Merged
merged 1 commit into from
Feb 7, 2024
Merged

Conversation

SSIRKC
Copy link
Contributor

@SSIRKC SSIRKC commented Feb 5, 2024

Hello Country Risk team,

as already announced we are moving the assessments to GitHub.

Please note that we have not published open severe findings if there are any.

This assessment file is a requirement for all quality gates and has to be provided as proof.

@szymonkowalczykzf please add the dataflow diagram in a separate pull request.

Assessment Creation.
@SSIRKC
Copy link
Contributor Author

SSIRKC commented Feb 5, 2024

@szymonkowalczykzf please also have a look as from your point of view.

@SSIRKC SSIRKC requested a review from fabiodmota February 5, 2024 12:56
Copy link
Contributor

@fabiodmota fabiodmota left a comment

Choose a reason for hiding this comment

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

Just checked the info added, anything you need from my side just mention or contact me

@SSIRKC
Copy link
Contributor Author

SSIRKC commented Feb 5, 2024

Just checked the info added, anything you need from my side just mention or contact me

Hi @fabiodmota,

can you automerge if you are okay with the file? :)

@fabiodmota fabiodmota requested a review from SujitMBRDI February 6, 2024 16:18
@fabiodmota
Copy link
Contributor

Just checked the info added, anything you need from my side just mention or contact me

Hi @fabiodmota,

can you automerge if you are okay with the file? :)

Yes , @SujitMBRDI is a commiter i wil ask him to merge this

@SujitMBRDI SujitMBRDI merged commit 59a9912 into eclipse-tractusx:main Feb 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants