Skip to content

[APPSEC-61865] Fix AppSec component initialization when crashing#5768

Merged
Strech merged 1 commit into
masterfrom
appsec-61865-fix-appsec-component-initialization
May 18, 2026
Merged

[APPSEC-61865] Fix AppSec component initialization when crashing#5768
Strech merged 1 commit into
masterfrom
appsec-61865-fix-appsec-component-initialization

Conversation

@Strech

@Strech Strech commented May 18, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Handles Exception-top level errors in AppSec component.

Motivation:

With Serverless team we discover that when AppSec failed to initialize it will crash host application as unhandled error bubbles up.

Change log entry

Yes. AppSec: Prevent crashing host application on initialization failure.

Additional Notes:

None.

How to test the change?

CI

@Strech
Strech requested a review from a team as a code owner May 18, 2026 10:52
@dd-octo-sts dd-octo-sts Bot added the appsec Application Security monitoring product label May 18, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5e737b2431

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/datadog/appsec/component.rb
@datadog-datadog-prod-us1-2

datadog-datadog-prod-us1-2 Bot commented May 18, 2026

Copy link
Copy Markdown

Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 97.14% (-0.01%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 5e737b2 | Docs | Datadog PR Page | Give us feedback!

Comment thread lib/datadog/appsec/component.rb
@Strech
Strech merged commit 5b93791 into master May 18, 2026
586 checks passed
@Strech
Strech deleted the appsec-61865-fix-appsec-component-initialization branch May 18, 2026 12:53
@dd-octo-sts dd-octo-sts Bot added this to the 2.34.0 milestone May 18, 2026
@Strech Strech mentioned this pull request May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

appsec Application Security monitoring product

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants