Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix sharkjack.sh #3

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

Fix sharkjack.sh #3

wants to merge 1 commit into from

Conversation

MrMan1994
Copy link

Fix shark jack interface not getting detected. Grepping the ip and invert grepping (grep -v) the same ip cannot return anything (unless -A is set to greater than 1), thus $IFACE is always empty.

Fix shark jack interface not getting detected. Grepping the ip and invert grepping (grep -v) the same ip cannot return anything (unless -A is set to greater than 1), thus $IFACE is always empty.
@tmossey
Copy link

tmossey commented Sep 17, 2019

Unfortunately I'm not getting the same results.

The current version returns:

$ ip route show to match 172.16.24.1 2>/dev/null| grep -i 172.16.24.1 | cut -d ' ' -f3 | grep -v 172.16.24.1

enp0s25

which is the intended result.

However this change returns:

$ ip route show to match 172.16.24.1 2>/dev/null| grep -i 172.16.24.1 | cut -d ' ' -f3

172.16.24.1
enp0s25

Here $IFACE isn't empty in either, but the current version results in the desired return value.

@MrMan1994
Copy link
Author

Alright, it seems as if I'm missing a route as only the IP is shown. Though I wonder how I got it to work in the first place... But how do I add that route? I can't get it to work neither ways anymore...

@foxtrot foxtrot assigned foxtrot and unassigned tmossey Jan 28, 2020
@foxtrot foxtrot added the enhancement New feature or request label Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants