Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix dupicated requests on linux #992

Merged
merged 1 commit into from
Aug 12, 2021
Merged

Fix dupicated requests on linux #992

merged 1 commit into from
Aug 12, 2021

Conversation

buger
Copy link
Owner

@buger buger commented Aug 12, 2021

Issue was introduced while fixing windows c9274ac

Added exception for Windows, which by default allows interfaces without IPs.
Interface name check moved higher, so if interface namee or IP match, rest of check will be ignored.

Additionally windows npcap loopback mechanism can now be picked by specifying 127.0.0.1 or loopback IP.

Fix #989

Issue was introduced while fixing windows c9274ac

Added exception for Windows, which by default allows interfaces without IPs.
Interface name check moved higher, so if interface namee or IP match, rest of check will be ignored.

Additionally windows npcap loopback mechanism can now be picked by specifying 127.0.0.1 or loopback IP.

Fix #989
@sonarcloud
Copy link

sonarcloud bot commented Aug 12, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@buger buger merged commit 42d8990 into master Aug 12, 2021
@buger buger deleted the fix/989-duplicate branch August 12, 2021 19:50
Copy link
Owner Author

buger commented Aug 12, 2021

Quality gate Quality gate: Passed

No issues Everything looks good
Details

@buger buger mentioned this pull request Aug 12, 2021
xingren23 referenced this pull request in xingren23/goreplay Sep 4, 2021
Issue was introduced while fixing windows buger@c9274ac

Added exception for Windows, which by default allows interfaces without IPs.
Interface name check moved higher, so if interface namee or IP match, rest of check will be ignored.

Additionally windows npcap loopback mechanism can now be picked by specifying 127.0.0.1 or loopback IP.

Fix #989
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.

Released gor1.3 forwards the same request twice on Linux
1 participant