Skip to content

Add track() API for custom event tracking - #360

Open
timokoessler wants to merge 1 commit into
mainfrom
custom-event-tracking
Open

Add track() API for custom event tracking#360
timokoessler wants to merge 1 commit into
mainfrom
custom-event-tracking

Conversation

@timokoessler

Copy link
Copy Markdown
Member

No description provided.

context.getMethod(),
context.getRemoteAddress(),
context.getHeader("user-agent"),
context.getUrl(),

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Medium - Custom tracking events exfiltrate full request URLs, including query strings

The new track() path builds its payload from context.getUrl(), so every custom event includes the full request URL rather than just the normalized route. In the Spring MVC integration that URL is explicitly reconstructed with the raw query string, which means applications that instrument benign flows such as password-reset or invite handling will now send any reset tokens, invite codes, or other query-parameter secrets to Aikido on every tracked event. This expands data collection beyond attack reporting and contradicts the documentation, which says track only auto-captures IP address, user agent, and current user.

Suggested change
context.getUrl(),
null,

More info - Reply on this comment to give feedback or ignore the issue.

@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant