Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG]: ModuleNotFoundError for ddtrace.appsec._iast._taint_tracking._native #12021

Open
benmatselby opened this issue Jan 22, 2025 · 6 comments
Assignees
Labels
ASM Application Security Monitoring bug

Comments

@benmatselby
Copy link

Tracer Version(s)

2.19*

Python Version(s)

Python 3.10.14

Pip Version(s)

pip 24.3.1

Bug Report

When we install ddtrace on any versions above 2.18.2 we get this error on MacOS. This is fine on Linux operating systems.

Reproduction Code

When we install any version from 2.19 onwards on MacOS

pip install ddtrace

Error Logs

We get this error so our application cannot load

/env/lib/python3.10/site-packages/ddtrace/appsec/_iast/_taint_tracking/__init__.py", line 1, in <module>
    from ddtrace.appsec._iast._taint_tracking._native import ops  # noqa: F401
ModuleNotFoundError: No module named 'ddtrace.appsec._iast._taint_tracking._native'

Libraries in Use

No response

Operating System

Darwin Kernel Version 24.2.0

@colinchilds-canopy
Copy link

I'm also experiencing this issue

@jclerman
Copy link

Also seen on: Darwin Kernel Version 24.3.0 (for what it's worth).

@chadhamre
Copy link

Same

1 similar comment
@johnnymetz
Copy link

Same

@gnufede
Copy link
Member

gnufede commented Feb 13, 2025

Hello everyone,

I'm trying to reproduce the issue, does anyone have a small snippet to reproduce it?

@benmatselby
Copy link
Author

Hey @gnufede I can replicate it with this snippet:

import requests
from ddtrace import patch_all

patch_all(urllib3=True)

response = requests.get("http://www.google.com")

With these dependencies:

pip install ddtrace
pip install requests

avara1986 added a commit that referenced this issue Feb 13, 2025
…12306)

This reverts commit 8c9beec.

This should fix #12021

## Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] 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](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
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](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

---------

Co-authored-by: Alberto Vara <[email protected]>
@avara1986 avara1986 added the ASM Application Security Monitoring label Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ASM Application Security Monitoring bug
Projects
None yet
Development

No branches or pull requests

7 participants