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

Update all dependencies #3247

Merged
merged 4 commits into from
May 20, 2024
Merged

Update all dependencies #3247

merged 4 commits into from
May 20, 2024

Conversation

nextgens
Copy link
Contributor

@nextgens nextgens commented May 6, 2024

What type of PR?

bug-fix

What does this PR do?

Update all dependencies including snappymail to 2.36.1

Related issue(s)

Prerequisites

Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

  • In case of feature or enhancement: documentation updated accordingly
  • Unless it's docs or a minor change: add changelog entry file.

@nextgens nextgens added the type/security Related to security label May 6, 2024
Copy link
Contributor

mergify bot commented May 6, 2024

Thanks for submitting this pull request.
Bors-ng will now build test images. When it succeeds, we will continue to review and test your PR.

bors try

Note: if this build fails, read this.

bors-mailu bot added a commit that referenced this pull request May 6, 2024
@@ -71,6 +71,7 @@ sp.disable_function.function("include").drop()

# Prevent `system`-related injections
sp.disable_function.function("system").param("command").value_r("[$|;&`\\n\\(\\)\\\\]").drop();
sp.disable_function.function("exec_shell").filename_r("/var/www/snappymail/snappymail/v/[0-9]+\.[0-9]+\.[0-9]+/app/libraries/snappymail/gpg/base.php").allow();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This could be tightened further... but it seems to break / need updating every update so I have made it looser than it has to be.

@nextgens nextgens added this to the 2024.x milestone May 6, 2024
@bors-mailu
Copy link
Contributor

bors-mailu bot commented May 6, 2024

try

Build failed:

@nextgens
Copy link
Contributor Author

nextgens commented May 6, 2024

bors try

bors-mailu bot added a commit that referenced this pull request May 6, 2024
@bors-mailu
Copy link
Contributor

bors-mailu bot commented May 6, 2024

try

Build failed:

@nextgens
Copy link
Contributor Author

nextgens commented May 6, 2024

bors try

bors-mailu bot added a commit that referenced this pull request May 6, 2024
@bors-mailu
Copy link
Contributor

bors-mailu bot commented May 6, 2024

try

Build failed:

@nextgens
Copy link
Contributor Author

nextgens commented May 6, 2024

bors try

bors-mailu bot added a commit that referenced this pull request May 6, 2024
@bors-mailu
Copy link
Contributor

bors-mailu bot commented May 6, 2024

try

Build succeeded:

@nextgens
Copy link
Contributor Author

@mergify.io rebase

@nextgens
Copy link
Contributor Author

bors try

bors-mailu bot added a commit that referenced this pull request May 20, 2024
@nextgens nextgens added type/enhancement Enhances existing functionality and removed type/security Related to security labels May 20, 2024
@bors-mailu
Copy link
Contributor

bors-mailu bot commented May 20, 2024

try

Build succeeded:

Copy link
Contributor

@ghostwheel42 ghostwheel42 left a comment

Choose a reason for hiding this comment

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

Let's get some wider testing feedback.

Copy link
Contributor

mergify bot commented May 20, 2024

bors r+

@bors-mailu
Copy link
Contributor

bors-mailu bot commented May 20, 2024

Build succeeded:

@bors-mailu bors-mailu bot merged commit 6e08e59 into Mailu:master May 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Enhances existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants