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

Logs and recordings are difficult to correlate #180

Closed
obilodeau opened this issue Feb 5, 2020 · 2 comments · Fixed by #181
Closed

Logs and recordings are difficult to correlate #180

obilodeau opened this issue Feb 5, 2020 · 2 comments · Fixed by #181
Assignees
Labels
enhancement New feature or request

Comments

@obilodeau
Copy link
Member

There is no clear link between json events and a given recording file.

There are actually two problems:

  • Filename on their own mean nothing (can't go from filename to logs)
  • Logs don't contain a reference to a given replay filename (can't go from json events to replay file)
@obilodeau obilodeau added the enhancement New feature or request label Feb 5, 2020
@obilodeau obilodeau self-assigned this Feb 5, 2020
@Res260
Copy link
Collaborator

Res260 commented Feb 5, 2020

The simplest solution would be to append the id/friendlyname of the connection (ex charlie71846) to the name of the replay file imo. It should be an easy change

@obilodeau
Copy link
Member Author

Yes, I already have something almost working but I'm working on having the replay filename in the logs too.

The way my ELK dashboard is setup, using the nice "Connection report" you've made, I would hunt for the correct log files. Without that there is a potential collision on the sessionID if we collect over several IPs for months. Just to give you an idea we had 3208 unique sessions in the last 12 hours on a single IP (2.5M+ events total). Of course, filtering by date would be fine but why just not add it anyway ;) Makes searching in the filesystem quicker too.

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.

2 participants