neofetch icon indicating copy to clipboard operation
neofetch copied to clipboard

Fix hostname detection for obscuring FQDN.

Open tidux opened this issue 3 years ago • 3 comments

Description

POSIX or not, the current FQDN detection doesn't work on modern Linux systems. This fixes it as tested on FC35.

Features

Stops leaking my hostname information.

Issues

resolves https://github.com/dylanaraps/neofetch/issues/2094

tidux avatar Apr 09 '22 20:04 tidux

Thank you for your contribution!

This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.

HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.

Read the "Running Updated Original Neofetch" section for more info!

hykilpikonna avatar Aug 11 '22 19:08 hykilpikonna

Just realized a problem... not all linux distros come with the inetutils that provides the hostname command. Maybe we should fall back to using $HOSTNAME if hostname command is not found.

hykilpikonna avatar Aug 13 '22 15:08 hykilpikonna

I changed it so that it's compatible with systems without the hostname command. I also replaced cut with parameter expansion

image

hykilpikonna avatar Aug 13 '22 18:08 hykilpikonna