Send app_path and ignore_logs to the collector - #276
Open
unflxw wants to merge 2 commits into
Open
Conversation
|
鉁旓笍 All good! |
unflxw
commented
Aug 25, 2026
unflxw
force-pushed
the
send-app-path-and-ignore-logs
branch
from
August 25, 2026 16:08
b5dda91 to
754e0d3
Compare
The processor strips the app path from each backtrace line, and decides whether a frame belongs to the application by checking that its path is relative. Without the app path, every frame keeps its absolute path, so no line is recognized as the application's own.
unflxw
force-pushed
the
send-app-path-and-ignore-logs
branch
from
August 25, 2026 16:17
754e0d3 to
bb004b7
Compare
unflxw
added
the
bug
There was a problem hiding this comment.
Pull request overview
This PR updates the AppSignal Python integration鈥檚 OpenTelemetry resource attributes so the collector receives (1) the application root path needed for correct backtrace parsing and (2) the new ignore_logs configuration for log filtering in collector mode.
Changes:
- Add
appsignal.config.app_pathto the OpenTelemetryResourceso the collector/processor can correctly relativize and classify backtrace frames. - Add
ignore_logsconfiguration (env + options) and send it asappsignal.config.ignore_logsin the OpenTelemetryResource. - Extend config/resource tests and add changesets documenting both changes.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/test_config.py | Extends env/config parsing assertions and validates new OTel resource attributes (app_path, ignore_logs). |
| src/appsignal/opentelemetry.py | Adds app_path and ignore_logs to the OTel Resource attributes sent to the collector. |
| src/appsignal/config.py | Adds the ignore_logs option to config typing and loads it from APPSIGNAL_IGNORE_LOGS. |
| .changesets/send-the-app-path-to-the-collector.md | Documents the backtrace/app-frame fix in collector mode (patch). |
| .changesets/add-the-ignore-logs-option.md | Documents the new ignore_logs option and its collector-mode-only behavior (minor). |
馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Collector mode sends log records, and the collector filters out the ones matching this option when the resource carries it. The package had no option for it because agent mode sends no logs at all, which is also why it is not passed on to the agent.
unflxw
force-pushed
the
send-app-path-and-ignore-logs
branch
from
August 25, 2026 16:30
bb004b7 to
723fc30
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #273.
Fixes #275.
Send the app path to the collector
The processor strips the app path from each backtrace line, and decides
whether a frame belongs to the application by checking that its path is
relative. Without the app path, every frame keeps its absolute path, so
no line is recognized as the application's own.
Add the ignore_logs option
Collector mode sends log records, and the collector filters out the ones
matching this option when the resource carries it. The package had no
option for it because agent mode sends no logs at all, which is also why
it is not passed on to the agent.
The option is documented in https://github.com/appsignal/appsignal-docs/pull/192,
including that it only has an effect in collector mode.