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

fix: Fix security issue #84

Merged
merged 3 commits into from
Dec 12, 2023
Merged

fix: Fix security issue #84

merged 3 commits into from
Dec 12, 2023

Conversation

adkumar1
Copy link
Contributor

@adkumar1 adkumar1 commented Dec 1, 2023

  • There is an security issue CVE-2023-33202 . This PR will be resolve it.

dvasunin
dvasunin approved these changes Dec 1, 2023
View reviewed changes
Copy link
Contributor

@dvasunin dvasunin left a comment

Choose a reason for hiding this comment

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

LGTM

@dvasunin
Copy link
Contributor

dvasunin commented Dec 4, 2023

@amoldashwant @sachinargade123 please review

@adkumar1 adkumar1 force-pushed the veracode_security_fix_23-12 branch 2 times, most recently from 5a3751c to 0dff91b Compare December 5, 2023 13:31
@adkumar1 adkumar1 mentioned this pull request Dec 8, 2023
Merged
almadigabor
almadigabor reviewed Dec 12, 2023
View reviewed changes
Copy link
Contributor

@almadigabor almadigabor left a comment

Choose a reason for hiding this comment

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

See my comment.

DEPENDENCIES Outdated Show resolved Hide resolved
almadigabor
almadigabor approved these changes Dec 12, 2023
View reviewed changes
Copy link
Contributor

@almadigabor almadigabor left a comment

Choose a reason for hiding this comment

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

LGTM

@almadigabor almadigabor merged commit b409b23 into eclipse-tractusx:main Dec 12, 2023
3 checks passed
@almadigabor almadigabor deleted the veracode_security_fix_23-12 branch December 12, 2023 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dvasunin dvasunin dvasunin approved these changes

@almadigabor almadigabor almadigabor approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@adkumar1 @dvasunin @almadigabor @amoldashwant