Skip to content

Conversation

bwoebi
Copy link
Contributor

@bwoebi bwoebi commented Sep 29, 2023

What does this PR do? What is the motivation?

Traces might be missing, crashes might happen, appsec may not work properly, ... unless that line is added to SELinux config.

Merge instructions

  • Please merge after reviewing

Traces might be missing, crashes might happen, appsec may not work properly, ... unless that line is added to SELinux config.
@bwoebi bwoebi requested a review from a team as a code owner September 29, 2023 16:23
@github-actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

Copy link
Contributor

@aliciascott aliciascott left a comment

Choose a reason for hiding this comment

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

Hey @bwoebi , this looks good, I just had to fix the code block within the callout as standard code blocks won't work in callouts; we have to either just use plain ticks or the lang param, elected to just use ticks here to be consistent with the format in the other callouts on the page.

@aliciascott aliciascott merged commit e512ade into master Sep 29, 2023
@aliciascott aliciascott deleted the bob/selinux-php branch September 29, 2023 20:07
klara-kr pushed a commit that referenced this pull request Oct 2, 2023
* Leave a note about the httpd selinux restrictions

Traces might be missing, crashes might happen, appsec may not work properly, ... unless that line is added to SELinux config.

* fixing code block in callout

---------

Co-authored-by: aliciascott <alicia.scott@datadoghq.com>
Co-authored-by: Alicia Scott <aliciascott@users.noreply.github.com>
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