Skip to content

[httpquery] fix: add method to parse headers and key/value data (#92)#122

Merged
antoinemzs merged 7 commits intomainfrom
bugfix/92
Nov 28, 2025
Merged

[httpquery] fix: add method to parse headers and key/value data (#92)#122
antoinemzs merged 7 commits intomainfrom
bugfix/92

Conversation

@savacano28
Copy link
Contributor

@savacano28 savacano28 commented Nov 21, 2025

Proposed changes

  • Add methods to parse string to list for headers and key/value http queries
image

Testing Instructions

  1. Create an inject type: http-query post key/value
  2. Add header: Content-Type: application/x-www-form-urlencoded and format request data: msg=data
  3. Execute inject
  4. Now, update inject: try header empty and the same request data, execute
  5. Now try inject with http-query with post raw, execute
  6. Now try inject with http-query with put raw, execute

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

@savacano28 savacano28 self-assigned this Nov 21, 2025
@savacano28 savacano28 added the filigran team use to identify PR from the Filigran team label Nov 21, 2025
@savacano28 savacano28 marked this pull request as ready for review November 21, 2025 17:41
@savacano28 savacano28 marked this pull request as draft November 27, 2025 14:13
@antoinemzs antoinemzs changed the title [http-query] fix: add method to parse headers and key/value data (#92) [httpquery] fix: add method to parse headers and key/value data (#92) Nov 28, 2025
Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>
Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>
@antoinemzs antoinemzs marked this pull request as ready for review November 28, 2025 16:11
Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>
Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>
@antoinemzs antoinemzs merged commit 71073c8 into main Nov 28, 2025
6 checks passed
@antoinemzs antoinemzs deleted the bugfix/92 branch November 28, 2025 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OpenAEV Http Query Injector

2 participants