chore(iast): native str aspect

[juanjux]

Description

Move the str() aspect to full native (requisite for repr and format_value).

TODO:

• More tests with propagation, the other error types and other input types.
• Remove the Python aspect and wire this one.
• Check for leaks.
• Check performance.

Checklist

• PR author has checked that all the criteria below are met
• The PR description includes an overview of the change
• The PR description articulates the motivation for the change
• The change includes tests OR the PR description describes a testing strategy
• The PR description notes risks associated with the change, if any
• Newly-added code is easy to change
• The change follows the library release note guidelines
• The change includes or references documentation updates if necessary
• Backport labels are set (if applicable)

Reviewer Checklist

• Reviewer has checked that all the criteria below are met
• Title is accurate
• All changes are related to the pull request's stated goal
• Avoids breaking API changes
• Testing strategy adequately addresses listed risks
• Newly-added code is easy to change
• Release note makes sense to a user of the library
• If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment
• Backport labels are set in a manner that is consistent with the release branch maintenance policy

[github-actions]

CODEOWNERS have been resolved as:

ddtrace/appsec/_iast/_taint_tracking/Aspects/AspectStr.cpp              @DataDog/asm-python
ddtrace/appsec/_iast/_taint_tracking/Aspects/AspectStr.h                @DataDog/asm-python
ddtrace/appsec/_iast/_taint_tracking/tests/test_str_aspect.cpp          @DataDog/asm-python

[gnufede]

Are the two commas (, ,) intended? What does this do?

[avara1986]

Yes it is, you can check other aspects
https://github.com/DataDog/dd-trace-py/blob/main/ddtrace/appsec/_iast/_taint_tracking/Aspects/AspectIndex.cpp#L81

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-10-08 21:38:52

Comparing candidate commit 1aa41e0 in PR branch juanjux/native-str-aspect with baseline commit cd1753d in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 214 metrics, 2 unstable metrics.