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

Bump express-openapi-validator from 5.3.5 to 5.3.6 #9347

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

steven-sheehy
Copy link
Member

@steven-sheehy steven-sheehy commented Sep 13, 2024

Description:

  • Bump express-openapi-validator from 5.3.5 to 5.3.6 and remove override
  • Downgrade path-to-regexp in swagger-stats to the newly patched 6.3.0
  • Re-enable Open API validator middleware

Related issue(s):

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
@steven-sheehy steven-sheehy added security dependencies Type: Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Sep 13, 2024
@steven-sheehy steven-sheehy added this to the 0.114.0 milestone Sep 13, 2024
@steven-sheehy steven-sheehy requested a review from a team September 13, 2024 15:22
@steven-sheehy steven-sheehy self-assigned this Sep 13, 2024
Copy link

sonarcloud bot commented Sep 13, 2024

Copy link

codecov bot commented Sep 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.50%. Comparing base (2eb3c1f) to head (96f6523).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #9347   +/-   ##
=========================================
  Coverage     92.50%   92.50%           
  Complexity     7012     7012           
=========================================
  Files           903      903           
  Lines         29646    29647    +1     
  Branches       3739     3739           
=========================================
+ Hits          27425    27426    +1     
  Misses         1450     1450           
  Partials        771      771           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@xin-hedera xin-hedera left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@jascks jascks left a comment

Choose a reason for hiding this comment

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

LGTM

@steven-sheehy steven-sheehy merged commit 52f9454 into main Sep 13, 2024
33 checks passed
@steven-sheehy steven-sheehy deleted the bump-express-openapi-validator branch September 13, 2024 16:04
@steven-sheehy steven-sheehy changed the title Bump express-open-validator from 5.3.5 to 5.3.6 Bump express-openapi-validator from 5.3.5 to 5.3.6 Sep 16, 2024
bilyana-gospodinova pushed a commit that referenced this pull request Sep 16, 2024
* Bump express-open-validator from `5.3.5` to `5.3.6` and remove override
* Downgrade path-to-regexp in swagger-stats to the newly patched `6.3.0`
* Re-enable Open API validator middleware

Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Type: Pull requests that update a dependency file javascript Pull requests that update Javascript code security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants