Skip to content

Feature/sentry#452

Merged
Nikhil-Nandagopal merged 6 commits into
releasefrom
feature/sentry
Aug 28, 2020
Merged

Feature/sentry#452
Nikhil-Nandagopal merged 6 commits into
releasefrom
feature/sentry

Conversation

@Nikhil-Nandagopal

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread app/client/src/utils/AnalyticsUtil.tsx
@Nikhil-Nandagopal Nikhil-Nandagopal merged commit bdbc736 into release Aug 28, 2020
@Nikhil-Nandagopal Nikhil-Nandagopal deleted the feature/sentry branch August 28, 2020 08:34
subrata71 added a commit that referenced this pull request Jun 4, 2026
## Description

Upgrades `nodemailer` from **6.9.9 → 8.0.10** and `@types/nodemailer`
from **^6.4.17 → ^8.0.0** in the RTS package to resolve all 8 open
Dependabot security alerts.

### What changed
- `app/client/packages/rts/package.json`: bumped `nodemailer` to
`8.0.10`, `@types/nodemailer` to `^8.0.0`
- `app/client/yarn.lock`: regenerated

### Why 8.x (not 7.x)
The highest-severity alerts (#574, #575) require patches only available
in `>= 8.0.5`. To fix **all** alerts with a single version bump, 8.x is
required.

### Impact assessment
- **Blast radius**: RTS only — `nodemailer` is declared solely in
`app/client/packages/rts/package.json`
- **Usage**: Single file (`app/client/packages/rts/src/ctl/mailer.ts`)
using only `createTransport` + `sendMail` with basic SMTP options — core
APIs unchanged across major versions
- **Risk**: Minimal — the upgrade is a drop-in replacement for the usage
patterns in this codebase

### Dependabot alerts resolved
| Alert | Severity | Min patch | Fixed by |
|-------|----------|-----------|----------|
|
[#432](https://github.com/appsmithorg/appsmith/security/dependabot/432)
| medium | >= 7.0.7 | 8.0.10 |
|
[#433](https://github.com/appsmithorg/appsmith/security/dependabot/433)
| medium | >= 7.0.7 | 8.0.10 |
|
[#451](https://github.com/appsmithorg/appsmith/security/dependabot/451)
| high | >= 7.0.11 | 8.0.10 |
|
[#452](https://github.com/appsmithorg/appsmith/security/dependabot/452)
| high | >= 7.0.11 | 8.0.10 |
|
[#544](https://github.com/appsmithorg/appsmith/security/dependabot/544)
| low | >= 8.0.4 | 8.0.10 |
|
[#545](https://github.com/appsmithorg/appsmith/security/dependabot/545)
| low | >= 8.0.4 | 8.0.10 |
|
[#574](https://github.com/appsmithorg/appsmith/security/dependabot/574)
| medium | >= 8.0.5 | 8.0.10 |
|
[#575](https://github.com/appsmithorg/appsmith/security/dependabot/575)
| medium | >= 8.0.5 | 8.0.10 |

Fixes https://linear.app/appsmith/issue/APP-15271

Slack thread:
https://theappsmith.slack.com/archives/C09NG5BJ18S/p1780304878402439

## Automation

/ok-to-test tags="@tag.All"

## Communication

Should the DevRel and Marketing teams be informed?

- [x] No

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

## Chores

* Updated core project dependencies to their latest versions for
enhanced compatibility and improved stability.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/26889525244>
> Commit: 09278de
> <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=26889525244&attempt=1"
target="_blank">Cypress dashboard</a>.
> Tags: `@tag.All`
> Spec:
> <hr>Wed, 03 Jun 2026 15:03:06 UTC
<!-- end of auto-generated comment: Cypress test results  -->
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