mwan3: default route src ip should be preferred over interface ip when get_src_ip
Open
Nobody has claimed this yet.
bug
release/23.05
- Dominant language
- Makefile
- Stars
- 4.6k
- Forks
- 4k
- Avg merge
- 3d 12h
- Merged PRs (30d)
- 134
Description
Package Name
mwan3
Maintainer
@feckert
OpenWrt Version
23.05.4
OpenWrt Target/Subtarget
mediatek/filogic
Steps to Reproduce
- Set up a custom "wan" interface
Iwith its IP beingA; - Add a route with a src ip being
Bdifferent to the interface IPA(e.g.ip route add default via X src B dev I metric 1000); - Add the interface
Ito mwan3; - Navigate to the mwan3 dianostics page (/cgi-bin/luci/admin/status/mwan3/diagnostics);
- Select the interface
Iand run "ping track ip";
Actual Behaviour
The interface IP A is used as SRCIP for ping. And due to some configuration choices, ping sourcing from the interface IP won't work in my case.
Expected behaviour:
The src ip B listed in the default route is expected to be used as SRCIP.
Confirmation Checklist
- The package is maintained in this repository.
- I understand that issues related to the base OpenWrt repository or LuCI repository will be closed.
- I am reporting an issue for OpenWrt, not an unsupported fork.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the mwan3 diagnostics page entry point (/cgi-bin/luci/admin/status/mwan3/diagnostics) and trace the “ping track ip” handling for the selected interface. Compare how the interface IP A and the default route source IP B are selected; done means ping uses B as SRCIP when the route provides it.
Written by the indexing model from the issue text.
Assessment
- Domain
- networking
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100