Skip to content

Conversation

@PDT42
Copy link
Contributor

@PDT42 PDT42 commented Dec 10, 2025

Adding node specific snippets and sections to the revised security guide, originally added in #2223.

Copy link
Collaborator

@finkmanAtSap finkmanAtSap left a comment

Choose a reason for hiding this comment

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

Just some small things 👍

@PDT42 PDT42 mentioned this pull request Dec 11, 2025
@PDT42 PDT42 changed the title Adding node specific Snippets & Sections Adding node specific Snippets & Sections (CAP User) Dec 11, 2025
@BraunMatthias BraunMatthias marked this pull request as ready for review December 11, 2025 15:38
@renejeglinsky
Copy link
Contributor

We'd like to avoid adding more toggled content, as the usage of the toggle brings a lot of problems and is discouraged. Would you like to prepare the content or should I give it a try and propose a change?

@renejeglinsky
Copy link
Contributor

For the first part it would be somehow like this:
image

@BraunMatthias
Copy link
Contributor

We'd like to avoid adding more toggled content, as the usage of the toggle brings a lot of problems and is discouraged. Would you like to prepare the content or should I give it a try and propose a change?

please let's optimize on the target branch after having merged this branch - otherwise we'll have to face lots of merge conflicts in the next days.

@renejeglinsky
Copy link
Contributor

Agreed :)

@renejeglinsky renejeglinsky merged commit b4cada0 into revised-security-guide Dec 12, 2025
1 of 2 checks passed
@renejeglinsky renejeglinsky deleted the nodejs/revised-security-guide branch December 12, 2025 08:26
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.

5 participants