Skip to content

GUACAMOLE-313: Add "guaclog" utility for producing human-readable interpretations of keys pressed in session recordings#137

Merged
asfgit merged 10 commits intoapache:masterfrom
mike-jumper:add-guaclog
Jan 30, 2018
Merged

GUACAMOLE-313: Add "guaclog" utility for producing human-readable interpretations of keys pressed in session recordings#137
asfgit merged 10 commits intoapache:masterfrom
mike-jumper:add-guaclog

Conversation

@mike-jumper
Copy link
Copy Markdown
Contributor

This change adds a new utility, "guaclog", which is analogous to guacenc in that it interprets the contents of session recordings or protocol dumps. Unlike guacenc, the guaclog utility tracks the current state of keys pressed, producing a human-readable log of what has been typed within a session.

Note that Guacamole does not currently log keys within session recordings, and the under-development support for this must be manually enabled if a recording should contain key events. For this to be useful to someone wishing to audit a Guacamole session, logging of key events would need to have been manually enabled ahead of time, or the full raw contents of that session (captured via tcpdump, for example) would need to be available.

Copy link
Copy Markdown
Contributor

@necouchman necouchman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me, but, again, I think maybe @ceharris @flangelo or @jmuehlner should double-check my review.

Copy link
Copy Markdown
Contributor

@flangelo flangelo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@asfgit asfgit merged commit fdd17e3 into apache:master Jan 30, 2018
@mike-jumper mike-jumper deleted the add-guaclog branch January 30, 2018 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants