remo-lab
remo-lab
Hi! I would like to work on this issue. Could you please assign it to me? I have already set up my local environment and started exploring the code.
@Mayanknishad9 The exact function name handleIncomingConnection() doesn't exist, but the relevant function is: handleConnection() at line 304 in pkg/agent/proxy/proxy.go This handles connections accepted by the proxy server (accepted at line...
For the issue assignment, you may also check with @officialasishkumar if he wants to keep this issue open or proceed with the fix as completed. Let me know if anything...
Hi @Sarthak160 ,I’ve opened PR #3355 with a fix for the test summary showing incorrect execution time (it used to include the user-provided delay). The update ensures the summary now...
Hi @Sarthak160 and @gouravkrosx , I’ve raised a PR for this issue that extracts the timestamp logic into a getBackdateTimestamp helper, updates both replay paths to use it, and adds...
I’ve added one more small commit to handle an edge case in the delay calculation. Earlier, the delay was only excluded when the execution time was greater than the delay....
Hi @Sarthak160, I’ve updated the implementation based on Copilot’s suggestion. The delay is now subtracted only when it was actually applied by using a delayApplied flag. This prevents incorrect time...
Hi @Sarthak160 , @gouravkrosx I have implemented the copiltot suggeszted changes. whenever you get time , i`d appreciate a reviwew.