Skip to content

Mpt 4059 vulnerability fixes #10

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

Merged
10 commits merged into from
Oct 28, 2024
Merged

Mpt 4059 vulnerability fixes #10

10 commits merged into from
Oct 28, 2024

Conversation

ghost
Copy link

@ghost ghost commented Oct 28, 2024

Description

Fix vulnerabilities detected by CodeQL & Bandit which wont affect the code itself, since we can't test it fully yet

Related issue number

fixes Mpt #4059

Special notes

only easy fixes where made which wont require intensive testing or larger changes in other areas

Checklist

  • The pull request title is a good summary of the changes

@maxb-hystax
Copy link

Hi @Flo-Weikert!
Would you mind to open corresponding PR in the main repo after merging this?

Also, I you can deliver the diff/PR of other mentioned changes which you can't properly test atm, we'll check them on our side, at least running unit tests on our runners.

Thanks!

@ffaraone ffaraone self-requested a review October 28, 2024 14:04
@ghost ghost merged commit 4bcb664 into main Oct 28, 2024
6 of 14 checks passed
@ffaraone ffaraone deleted the MPT-4059_Vulnerability_Fixes branch November 20, 2024 09:45
ffaraone added a commit that referenced this pull request Nov 20, 2024
ffaraone pushed a commit that referenced this pull request Nov 22, 2024
ffaraone pushed a commit that referenced this pull request Feb 26, 2025
- increased max pod per node to 250
- increased ingress controller memory limit

## Description

<!-- Please give a summary of the changes. -->

## Related issue number

<!-- Please link a pull request to an issue by using "fixes #10" style
references to close the issue when this PR is merged. -->

## Special notes

<!-- Please provide additional information if required. -->

## Checklist

* [ ] The pull request title is a good summary of the changes
* [ ] Unit tests for the changes exist
* [ ] New and existing unit tests pass locally
ffaraone pushed a commit that referenced this pull request Feb 26, 2025
This reverts commit 16376e1.

## Description

<!-- Please give a summary of the changes. -->

## Related issue number

<!-- Please link a pull request to an issue by using "fixes #10" style
references to close the issue when this PR is merged. -->

## Special notes

<!-- Please provide additional information if required. -->

## Checklist

* [ ] The pull request title is a good summary of the changes
* [ ] Unit tests for the changes exist
* [ ] New and existing unit tests pass locally
This pull request was closed.
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