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.

Sign up for GitHub

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

Jump to bottom

adjust limiting the php-fpm port to specific containers again #3427

Merged
merged 1 commit into from
Oct 2, 2023
Merged

adjust limiting the php-fpm port to specific containers again #3427

merged 1 commit into from
Oct 2, 2023

Conversation

szaimen
Copy link
Collaborator

@szaimen szaimen commented Sep 27, 2023 •
edited
Loading

Close #3172

@szaimen szaimen added the 2. developing Work in progress label Sep 27, 2023
@szaimen szaimen added this to the next milestone Sep 27, 2023
@szaimen szaimen mentioned this pull request Sep 27, 2023
Closed
37 tasks
@szaimen szaimen force-pushed the enh/noid/adjust-limiting branch 2 times, most recently from 6886844 to d992ccd Compare September 29, 2023 16:41
@szaimen szaimen force-pushed the enh/noid/adjust-limiting branch from d992ccd to d82d345 Compare September 29, 2023 16:43
@szaimen szaimen added the enhancement New feature or request label Sep 29, 2023
@szaimen
adjust limiting the php-fpm port to specific containers again
f82affd 
Signed-off-by: Simon L <szaimen@e.mail.de>
@szaimen szaimen force-pushed the enh/noid/adjust-limiting branch from d82d345 to f82affd Compare September 29, 2023 17:15
@szaimen szaimen marked this pull request as ready for review September 29, 2023 17:16
@szaimen szaimen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 29, 2023
@szaimen szaimen requested a review from Zoey2936 September 29, 2023 17:17
Zoey2936
Zoey2936 reviewed Sep 29, 2023
View reviewed changes
Copy link
Collaborator

@Zoey2936 Zoey2936 left a comment

Choose a reason for hiding this comment

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

can't review, I have no idea

@szaimen
Copy link
Collaborator Author

szaimen commented Oct 2, 2023

can't review, I have no idea

all right

@szaimen szaimen merged commit 1426646 into main Oct 2, 2023
9 checks passed
@delete-merged-branch delete-merged-branch bot deleted the enh/noid/adjust-limiting branch October 2, 2023 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Zoey2936 Zoey2936 Zoey2936 left review comments

Assignees
No one assigned
Labels
3. to review Waiting for reviews enhancement New feature or request
Projects
None yet
Milestone
v7.4.0
Development

Successfully merging this pull request may close these issues.

Make mastercontainer and nextcloud container more secure by setting listen.allowed_clients
2 participants
@szaimen @Zoey2936