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

tls子命令可否支持ip过滤 #474

Closed
onlytiancai opened this issue Jan 31, 2024 · 6 comments · Fixed by #478
Closed

tls子命令可否支持ip过滤 #474

onlytiancai opened this issue Jan 31, 2024 · 6 comments · Fixed by #478
Assignees
Labels
enhancement New feature or request

Comments

@onlytiancai
Copy link

./ecapture tls 问下只有--port参数,没看到--host或--ip的参数,有计划支持吗?

  • 一般生产环境不会全量抓包,比如就专门指定客户端IP抓这个客户的包之类
  • 或者就是调试个网络请求啥的,类似 tcpdump -i eth0 host 192.168.0.1 and port 80
@cfc4n cfc4n added the enhancement New feature or request label Jan 31, 2024
@cfc4n cfc4n self-assigned this Jan 31, 2024
@cfc4n
Copy link
Member

cfc4n commented Jan 31, 2024

@Asphaltt 大神有个更好地解决方案 pcap filter ,未来会支持这个功能。

Expert @Asphaltt has a better solution pcap filter, which will be supported in the future.

@Asphaltt
Copy link
Contributor

Do you mean elibpcap library?

It will be nice to support pcap filter expression like tcpdump.

@cfc4n WDYT?

@cfc4n
Copy link
Member

cfc4n commented Jan 31, 2024

好像也只能如此了。 这周我测试github.com/Asphaltt/tc-dump在android上的兼容性。

It seems that this is the only way. This week, I tested the compatibility of github.com/Asphaltt/tc-dump on Android.

@Asphaltt
Copy link
Contributor

Asphaltt commented Feb 1, 2024

I tested the compatibility of github.com/Asphaltt/tc-dump on Android.

Great.

One more question: is --port=443 required? It will conflict with pcap-filter when there's tcp port 8080 in pcap-filter at the same time.

@cfc4n
Copy link
Member

cfc4n commented Feb 1, 2024

Well, the '--port' argument does conflict with 'pcap-fliter' and can be removed


嗯,--port参数确实会跟pcap-fliter产生冲突,可以去掉。

@Asphaltt
Copy link
Contributor

Asphaltt commented Feb 1, 2024

can be removed

Good. I'm on it.

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 a pull request may close this issue.

3 participants