build icon indicating copy to clipboard operation
build copied to clipboard

Improvement: Update MOTD to include IPv6 local addresses

Open SuperKali opened this issue 1 year ago • 3 comments

This PR include these changes:

  • Added functionality to display local IPv6 addresses in the MOTD.
  • Reverted: Default behavior now excludes interfaces matching 'br-*' and 'veth'.
  • This PR is referenced to another PR #7299, will be closed.

How Has This Been Tested?

Tested on my board:

For privacy reason I didn't include the WAN IP on the screenshot 👀

image

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [x] I have commented my code, particularly in hard-to-understand areas
  • [x] My changes generate no new warnings
  • [x] Any dependent changes have been merged and published in downstream modules

SuperKali avatar Sep 30 '24 21:09 SuperKali

Default behavior now excludes interfaces matching 'br-*' and 'veth'.

I don't think this is a good idea. If bridge is set, then there is IP address set.

igorpecovnik avatar Oct 02 '24 12:10 igorpecovnik

Default behavior now excludes interfaces matching 'br-*' and 'veth'.

I don't think this is a good idea. If bridge is set, then there is IP address set.

Ok, i will remove it from the commit

SuperKali avatar Oct 02 '24 13:10 SuperKali

tabs

I will check tomorrow :)

SuperKali avatar Oct 03 '24 20:10 SuperKali