Skip to content

Commit e1190c8

Browse files
chore: upgrade header filtering log messages to info level
- Change header filtering log messages from debug to info level - Provides better visibility into header forwarding behavior - Important for operational monitoring and troubleshooting
1 parent 1a4984f commit e1190c8

File tree

2 files changed

+17
-17
lines changed

2 files changed

+17
-17
lines changed

src/agentex/lib/core/services/adk/acp/acp.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -69,12 +69,12 @@ def _filter_headers(
6969

7070
# Pass-through behavior: if no config, forward all headers
7171
if config is None:
72-
logger.debug("No header filtering config found, passing through all %d headers", len(headers))
72+
logger.info("No header filtering config found, passing through all %d headers", len(headers))
7373
return headers
7474

7575
# Apply filtering based on allowlist
7676
if not config.allowed_headers:
77-
logger.debug("Empty allowlist in config, blocking all headers")
77+
logger.info("Empty allowlist in config, blocking all headers")
7878
return {}
7979

8080
filtered = {}
@@ -88,11 +88,11 @@ def _filter_headers(
8888

8989
if header_allowed:
9090
filtered[header_name] = header_value
91-
logger.debug("Allowed header: %s", header_name)
91+
logger.info("Allowed header: %s", header_name)
9292
else:
93-
logger.debug("Header '%s' not in allowlist, ignoring", header_name)
93+
logger.info("Header '%s' not in allowlist, ignoring", header_name)
9494

95-
logger.debug("Filtered %d headers from %d based on allowlist", len(filtered), len(headers))
95+
logger.info("Filtered %d headers from %d based on allowlist", len(filtered), len(headers))
9696
return filtered
9797

9898
async def task_create(

src/agentex/lib/types/acp.py

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ class CreateTaskParams(BaseModel):
2525
agent: The agent that the task was sent to.
2626
task: The task to be created.
2727
params: The parameters for the task as inputted by the user.
28-
extra_headers: Custom headers forwarded to this agent (filtered by manifest allowlist).
28+
request: Additional request context including headers forwarded to this agent.
2929
"""
3030

3131
agent: Agent = Field(..., description="The agent that the task was sent to")
@@ -34,9 +34,9 @@ class CreateTaskParams(BaseModel):
3434
None,
3535
description="The parameters for the task as inputted by the user",
3636
)
37-
extra_headers: dict[str, str] | None = Field(
37+
request: dict[str, Any] | None = Field(
3838
default=None,
39-
description="Custom headers forwarded to this agent (filtered by manifest allowlist)",
39+
description="Additional request context including headers forwarded to this agent",
4040
)
4141

4242

@@ -48,7 +48,7 @@ class SendMessageParams(BaseModel):
4848
task: The task that the message was sent to.
4949
content: The message that was sent to the agent.
5050
stream: Whether to stream the message back to the agentex server from the agent.
51-
extra_headers: Custom headers forwarded to this agent (filtered by manifest allowlist).
51+
request: Additional request context including headers forwarded to this agent.
5252
"""
5353

5454
agent: Agent = Field(..., description="The agent that the message was sent to")
@@ -60,9 +60,9 @@ class SendMessageParams(BaseModel):
6060
False,
6161
description="Whether to stream the message back to the agentex server from the agent",
6262
)
63-
extra_headers: dict[str, str] | None = Field(
63+
request: dict[str, Any] | None = Field(
6464
default=None,
65-
description="Custom headers forwarded to this agent (filtered by manifest allowlist)",
65+
description="Additional request context including headers forwarded to this agent",
6666
)
6767

6868

@@ -73,15 +73,15 @@ class SendEventParams(BaseModel):
7373
agent: The agent that the event was sent to.
7474
task: The task that the message was sent to.
7575
event: The event that was sent to the agent.
76-
extra_headers: Custom headers forwarded to this agent (filtered by manifest allowlist).
76+
request: Additional request context including headers forwarded to this agent.
7777
"""
7878

7979
agent: Agent = Field(..., description="The agent that the event was sent to")
8080
task: Task = Field(..., description="The task that the message was sent to")
8181
event: Event = Field(..., description="The event that was sent to the agent")
82-
extra_headers: dict[str, str] | None = Field(
82+
request: dict[str, Any] | None = Field(
8383
default=None,
84-
description="Custom headers forwarded to this agent (filtered by manifest allowlist)",
84+
description="Additional request context including headers forwarded to this agent",
8585
)
8686

8787

@@ -91,14 +91,14 @@ class CancelTaskParams(BaseModel):
9191
Attributes:
9292
agent: The agent that the task was sent to.
9393
task: The task that was cancelled.
94-
extra_headers: Custom headers forwarded to this agent (filtered by manifest allowlist).
94+
request: Additional request context including headers forwarded to this agent.
9595
"""
9696

9797
agent: Agent = Field(..., description="The agent that the task was sent to")
9898
task: Task = Field(..., description="The task that was cancelled")
99-
extra_headers: dict[str, str] | None = Field(
99+
request: dict[str, Any] | None = Field(
100100
default=None,
101-
description="Custom headers forwarded to this agent (filtered by manifest allowlist)",
101+
description="Additional request context including headers forwarded to this agent",
102102
)
103103

104104

0 commit comments

Comments
 (0)