mirror of
https://github.com/ProxymanApp/atlantis.git
synced 2026-04-26 08:26:04 +03:00
[GH-ISSUE #71] Send request to Proxyman on start of request #48
Labels
No labels
Done
Done
Windows
bug
bug
bug
enhancement
enhancement
enhancement
good first issue
hacktoberfest
pull-request
wontfix
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/atlantis#48
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @djbe on GitHub (May 12, 2021).
Original GitHub issue: https://github.com/ProxymanApp/atlantis/issues/71
Originally assigned to: @NghiaTranUIT on GitHub.
Based on https://github.com/ProxymanApp/atlantis/issues/28#issuecomment-840056741:
Right now, Atlantis only sends a message to Proxyman when a requests finishes (success or failure). But, since
TrafficPackagehave an ID (here), nothing should stop us from sending the info before the start of a call, and then when a call finishes, send the updated package and have Proxyman update the call/row as needed.Note that this is how Bagel works, which Atlantis is (kinda) based on:
Right now, if we modify Atlantis to send the traffic package at the start (inside
injectorSessionDidCallResume), the call will appear twice in Proxyman: one "active" incomplete call, and one "completed" call with all the data (see screenshot).Todo:
@NghiaTranUIT commented on GitHub (May 13, 2021):
I'm afraid it's too difficult to change the logic from Atlantis and Proxyman, which is live on the Production.
I would highly recommend that you might update your Kotlin code to send a message to Proxyman in a single package (when you have a Request and a Response)
@djbe commented on GitHub (May 13, 2021):
Note that for #28, we've indeed changed the logic for now to only send it once. I made a separate issue for this, as it could be an improvement in general, especially for long lived network operations.
While it would indeed require changes to Proxyman itself, it wouldn't be the first time Proxyman or Atlantis require a new (or minimum) version of each other? See for example the latest Atlantis 1.9.0 version, which requires Proxyman 2.24 or higher, because of the addition of websocket logging.