Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

LocalIP (macOS): improve detection #2362

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CarterLi
Copy link

When a VPN session is enabled, the default route interface is changed to utun*, however ipconfig doesn't support it for some reason

Use ifconfig instead


image

When a VPN session is enabled, the default route interface is changed to `utun*`, however `ipconfig` doesn't support it for some reason

Use `ifconfig` instead
@CarterLi
Copy link
Author

Ad: Use fastfetch for more powerful and faster Local IP detection

image

@CarterLi
Copy link
Author

@hykilpikonna 你为什么可以在别人的仓库里批准PR?

@hykilpikonna
Copy link

@hykilpikonna 你为什么可以在别人的仓库里批准PR?

This is a standard feature on Github, anyone can approve PR in any public repository.

Sorry for the delay, I'll merge this to HyFetch after I test it on my macOS system later.

@CarterLi
Copy link
Author

This is a standard feature on Github, anyone can approve PR in any public repository.

How?

@hykilpikonna
Copy link

How?

  1. Go to an open PR in a public repository.
  2. Click the "Files changed" tab
  3. Click the green "Review changes" dropdown on top right
  4. Click "Approve"

@hykilpikonna
Copy link

Found one issue: If the reverse DNS lookup for the gateway takes too long, the route get 1 command will be stuck for 30 seconds before it timeouts.

image

hykilpikonna added a commit to hykilpikonna/hyfetch that referenced this pull request Aug 15, 2023
@hykilpikonna
Copy link

IP local address works!

image

hykilpikonna added a commit to hykilpikonna/hyfetch that referenced this pull request Aug 15, 2023
Upstream PR: dylanaraps#2362
Thanks to @CarterLi

Co-authored-by: 李通洲 <zhangsongcui3371@sina.com>
@hykilpikonna
Copy link

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!

@CarterLi
Copy link
Author

Please don't do that. I'm submitting this PR is for advertising my work. I don't care if it gets merged or not

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
Copy link

Please don't do that. I'm submitting this PR is for advertising my work. I don't care if it gets merged or not

I apologize for assuming that everyone who opens a pull request wants to contribute to the project, and it's unfortunate that you don't care whether your contribution is accepted. I'm grateful for your contribution regardless. 😃

hayatehay added a commit to nmimusic/unifetch that referenced this pull request Aug 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants