Skip to content

Commit

Permalink
don't publish full query text on TAP events
Browse files Browse the repository at this point in the history
  • Loading branch information
fajpunk committed Dec 16, 2024
1 parent 50b9b9f commit 7b39065
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 5 deletions.
1 change: 0 additions & 1 deletion src/mobu/events.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,6 @@ class TapQuery(EventBase):
success: bool
duration: timedelta | None
sync: bool
query: str


class Events(EventMaker):
Expand Down
1 change: 0 additions & 1 deletion src/mobu/services/business/tap.py
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,6 @@ async def execute(self) -> None:
success=success,
duration=sw.elapsed,
sync=self.options.sync,
query=query,
**self.common_event_attrs(),
)
)
Expand Down
1 change: 0 additions & 1 deletion tests/business/tapqueryrunner_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ async def test_run(client: AsyncClient, respx_mock: respx.Router) -> None:
"business": "TAPQueryRunner",
"duration": NOT_NONE,
"flock": "test",
"query": NOT_NONE,
"success": True,
"sync": True,
"username": "bot-mobu-testuser1",
Expand Down
2 changes: 0 additions & 2 deletions tests/business/tapquerysetrunner_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,6 @@ async def test_run(client: AsyncClient, respx_mock: respx.Router) -> None:
"business": "TAPQuerySetRunner",
"duration": NOT_NONE,
"flock": "test",
"query": NOT_NONE,
"success": True,
"sync": True,
"username": "bot-mobu-testuser1",
Expand Down Expand Up @@ -196,7 +195,6 @@ async def test_failure(
"business": "TAPQuerySetRunner",
"duration": NOT_NONE,
"flock": "test",
"query": NOT_NONE,
"success": False,
"sync": True,
"username": "bot-mobu-testuser1",
Expand Down

0 comments on commit 7b39065

Please sign in to comment.