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

Inaccurate/Confusing Diagrams #34

Closed
andrewstucki opened this issue Apr 19, 2022 · 1 comment · Fixed by #36
Closed

Inaccurate/Confusing Diagrams #34

andrewstucki opened this issue Apr 19, 2022 · 1 comment · Fixed by #36
Labels
good first issue Good for newcomers

Comments

@andrewstucki
Copy link

Just ran across this project, looks neat. Taking a look at the underlying implementation, this works as a program that just hooks common shared library encryption routines via uprobes and events with the unencrypted data as those routines are invoked. That makes sense.

The README diagrams however, are kind of odd. They:

  1. Have libpcap in them, which, as far as I can tell doesn't appear to be leveraged anywhere in this project
  2. Within libpcap shows TC and XDP boxes, almost implying that this works with network-level packet capture

Neither of those seem accurate, since installing a system-wide uprobe hook into say, the openssl library, will event on data even if it's being encrypted/decrypted locally on a filesystem and doing 0 with the network.

I think it'd be helpful to update the diagrams to better illustrate the uprobe hooks without any reference to libpcap or the network just for people wondering how this actually works.

@cfc4n cfc4n added the good first issue Good for newcomers label Apr 20, 2022
cfc4n added a commit that referenced this issue Apr 20, 2022
Signed-off-by: CFC4N <cfc4n.cs@gmail.com>
@cfc4n cfc4n linked a pull request Apr 20, 2022 that will close this issue
@cfc4n cfc4n closed this as completed in #36 Apr 20, 2022
cfc4n added a commit that referenced this issue Apr 20, 2022
images : fix #34 Inaccurate/Confusing Diagrams
@cfc4n
Copy link
Member

cfc4n commented Apr 20, 2022

I agree, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants