Last December, I had an issue with ICMP
This could just be my own confusement, but isn’t Netbird dependent on being able to maskerade the connection for machines that do not have netbird behind them? If you look in the firewall logs, does it display any IMCP logs to attempted locations?
My best guess here is that you are pinging the netbird agent no matter what IP you put in there. I could also be wrong, as it’s just a guess here. Since it has to go through the agent anyway to get into the subnet.
The issue was fixed
main ← forwarder-ping
opened 11:45AM - 15 Nov 25 UTC
## Describe your changes
- Use the ping binary if we cannot create raw socket… s
- Bump gvisor/netstack to fix duplicate echo replies (this also bumps the min Go version)
- Bump wireguard-go fork to work with the new gvisor
- Adjust forwarder to work with new netstack code
- Adjust ice_bind to work with new wireguard-go code
- Fix a bug where waiting for echo replies would block all other packets
Depends on https://github.com/netbirdio/wireguard-go/pull/12
## Issue ticket number and link
## Stack
### Checklist
- [x] Is it a bug fix
- [ ] Is a typo/documentation fix
- [x] Is a feature enhancement
- [ ] It is a refactor
- [ ] Created tests that fail without the change (if possible)
> By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md).
## Documentation
Select exactly one:
- [ ] I added/updated documentation for this change
- [x] Documentation is **not needed** for this change (explain why)
### Docs PR URL (required if "docs added" is checked)
Paste the PR link from https://github.com/netbirdio/docs here:
https://github.com/netbirdio/docs/pull/__
## Summary by CodeRabbit
* **New Features**
* Asynchronous ICMP echo handling with startup raw-socket capability detection, rate-limited system-ping fallback, and synthesized echo replies.
* UDP handling now reports explicit success/failure outcomes.
* Receiver creation broadened to support non-IPv4 batch readers.
* Added an extended warn-level logging helper.
* **Bug Fixes / Stability**
* MTU access made atomic to reduce races and improve stability.
* **Chores**
* Bumped several core dependencies and module replacements.
<sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub>
… but the issue has regressed.
@Vik would you mind taking a peek again?@acaronmd think you could create a debug code for the staff to work with? Try doing your ICMP request during the debug so they have ample information to work with
