• General syntax update consequent to bump to 1.18
• Check if the reader function can be simplified with ReusableReader

closes #30

Possible Explaination

Each unit test created a new web server (buggy http) on same port . but each of them ran on goroutines .
Instead of failing to bind to port . it could have caused race condition (at lower level). . Making all unit test use same server for testing fixed the issue

Changes

• Simplified logic (related to request body reuse)
• Improvements and efficiency
• Added more unit test to avoid similar situations in future

Verified with projectdiscovery/nuclei#2924 . Works as expected !!

Excellent work - It seems much more uniform and efficient now, additionally the underlying http.Transport won't be locked into I/O waiting operations. Not related to this PR, but I'd also like to add these changes related to the lint-test.yml action:

• Add checkout v3
• Add actions/setup-go@v3 before calling golangci-lint-action (a few times, the analyzer got stuck due to missing go environment)

lgtm - merging