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

New CS Microservice security #491

Merged
merged 7 commits into from Oct 13, 2020
Merged

New CS Microservice security #491

merged 7 commits into from Oct 13, 2020

Conversation

@AlexB1986
Copy link
Contributor

@AlexB1986 AlexB1986 commented Sep 23, 2020

This PR covers issue #119 .

AlexB1986 added 4 commits Sep 23, 2020
@AlexB1986 AlexB1986 requested review from jmanico and mackowski as code owners Sep 23, 2020
@mackowski
Copy link
Collaborator

@mackowski mackowski commented Sep 23, 2020

@AlexB1986 it looks awesome after first quick look. I will review it this week. Thank you!

Copy link
Collaborator

@mackowski mackowski left a comment

I really like this CS, I think that this is awesome.
Please add table of contents after Introduction and I will approve it.
One comment for the future development - it would be good to add example other than Netflix
@jmanico please review this work.

AlexB1986 added 2 commits Oct 5, 2020
@AlexB1986
Copy link
Contributor Author

@AlexB1986 AlexB1986 commented Oct 5, 2020

I really like this CS, I think that this is awesome.
Please add table of contents after Introduction and I will approve it.
One comment for the future development - it would be good to add example other than Netflix
@jmanico please review this work.

@mackowski, thanks for your review and feedback. I added the table of contents. In future I will add more examples other than Netflix.

@AlexB1986 AlexB1986 requested a review from mackowski Oct 5, 2020
@jmanico
jmanico approved these changes Oct 6, 2020
Copy link
Member

@jmanico jmanico left a comment

This is fantastic and I love the direction.

@jmanico
Copy link
Member

@jmanico jmanico commented Oct 6, 2020

Copy link
Collaborator

@mackowski mackowski left a comment

Awesome job :)

@mackowski mackowski merged commit 1cac62d into OWASP:master Oct 13, 2020
3 checks passed
3 checks passed
link-check
Details
lint
Details
Publishing Check
Details
c0r0n3r added a commit to c0r0n3r/CheatSheetSeries that referenced this pull request Oct 24, 2020
* Create Microservies_security.md

* Add files via upload

* Update Microservies_security.md

* Update Microservies_security.md

* Update Microservies_security.md

* Update Microservies_security.md

* Update Microservies_security.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants