2 Low
CVSS3
Attack Vector
NETWORK
Attack Complexity
HIGH
Privileges Required
HIGH
User Interaction
REQUIRED
Scope
UNCHANGED
Confidentiality Impact
LOW
Integrity Impact
NONE
Availability Impact
NONE
CVSS:3.1/AV:N/AC:H/PR:H/UI:R/S:U/C:L/I:N/A:N
6.5 Medium
AI Score
Confidence
High
0.0004 Low
EPSS
Percentile
10.4%
Sentry’s Slack integration incorrectly records the incoming request body in logs. This request data can contain sensitive information, including the deprecated Slack verification token. With this verification token, it is possible under specific configurations, an attacker can forge requests and act as the Slack integration.
The request body is leaked in log entries matching event == "slack.*" && name == "sentry.integrations.slack" && request_data == *
. The deprecated slack verification token, will be found in the request_data.token
key.
Example event:
{
"name": "sentry.integrations.slack",
"level": "info",
"event": "slack.event.message", # This could be any of the `slack.*` events
"request_data": {
# Other keys are omitted for brevity
"token": "<MyDeprecatedSlackVerificationToken>",
}
}
slack.signing-secret
in your self-hosted configuration, then the legacy verification token is not used to verify the webhook payload. It is ignored.> ⚠️ Sentry’s support for validating Slack requests via the legacy verification token will be deprecated in version 24.7.0.
Set the slack.signing-secret
instead of slack.verification-token
. The signing secret is Slack’s recommended way of authenticating webhooks.
By having slack.singing-secret
set, Sentry self-hosted will no longer use the verification token for authentication of the webhooks, regardless of whether slack.verification-token
is set or not.
The deprecated Slack verification token is leaked in log levels of INFO
and ERROR
in the Slack integration. If the self-hosted instance is unable to be upgraded or re-configured to use the slack.signing-secret
, the logging configuration can be adjusted to not generate logs from the integration. The default logging configuration can be found in src/sentry/conf/server.py
. Services should be restarted once the configuration change is saved.
Below you’ll find an example of the configuration adjustments necessary to remove the Slack integration logs:
# src/sentry/conf/server.py
...
LOGGING: LoggingConfig = {
...
handlers: {
# the line below already exists in the default configuration
"null": {"class": "logging.NullHandler"},
...
},
"loggers": {
"sentry.integrations.slack": {
"handlers": ["null"], # route logs to null handler
"level": "CRITICAL", # prevent generation of logs a lower levels (ex. ERROR and INFO)
},
...
},
}
api.slack.com/authentication/verifying-requests-from-slack#app-management-updates
api.slack.com/authentication/verifying-requests-from-slack#deprecation
api.slack.com/authentication/verifying-requests-from-slack#regenerating
develop.sentry.dev/integrations/slack
github.com/advisories/GHSA-c2g2-gx4j-rj3j
github.com/getsentry/sentry/blob/17d2b87e39ccd57e11da4deed62971ff306253d1/src/sentry/conf/server.py#L1307
github.com/getsentry/sentry/pull/70508
github.com/getsentry/sentry/security/advisories/GHSA-c2g2-gx4j-rj3j
nvd.nist.gov/vuln/detail/CVE-2024-35196
2 Low
CVSS3
Attack Vector
NETWORK
Attack Complexity
HIGH
Privileges Required
HIGH
User Interaction
REQUIRED
Scope
UNCHANGED
Confidentiality Impact
LOW
Integrity Impact
NONE
Availability Impact
NONE
CVSS:3.1/AV:N/AC:H/PR:H/UI:R/S:U/C:L/I:N/A:N
6.5 Medium
AI Score
Confidence
High
0.0004 Low
EPSS
Percentile
10.4%