Skip to content

chore: Pin SES - #4151

Merged
FrederikBolding merged 3 commits into
mainfrom
fb/pin-ses
Oct 9, 2026
Merged

FrederikBolding merged 3 commits into
mainfrom
fb/pin-ses

Conversation

@FrederikBolding

@FrederikBolding FrederikBolding commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

Pins SES to 2.2.0 since >=2.3.0 has a bug that prevents it from running in some versions of Chrome.

@socket-security

socket-security Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedses@​2.3.0 ⏵ 2.2.088 -1210010093 +1100

View full report

@socket-security

socket-security Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Ignoring alerts on:

  • ses@2.2.0

View full report

@FrederikBolding

Copy link
Copy Markdown
Member Author

@SocketSecurity ignore npm/ses@1.15.0

False positive + trusted author.

@codecov

codecov Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.65%. Comparing base (f3affd2) to head (69bedac).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4151   +/-   ##
=======================================
  Coverage   98.65%   98.65%           
=======================================
  Files         429      429           
  Lines       12523    12523           
  Branches     1979     1979           
=======================================
  Hits        12355    12355           
  Misses        168      168           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@FrederikBolding

Copy link
Copy Markdown
Member Author

@SocketSecurity ignore npm/ses@2.2.0

Still a false positive.

@FrederikBolding
FrederikBolding marked this pull request as ready for review October 9, 2026 12:37
@FrederikBolding
FrederikBolding requested a review from a team as a code owner October 9, 2026 12:37
@FrederikBolding
FrederikBolding added this pull request to the merge queue Oct 9, 2026
Merged via the queue into main with commit 704e593 Oct 9, 2026
130 checks passed
@FrederikBolding
FrederikBolding deleted the fb/pin-ses branch October 9, 2026 12:39
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