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

Proposal: refactor logging, capture for dashboard recordings, output to file, serialize browser console logs #2078

Closed
brian-mann opened this issue Jul 1, 2018 · 3 comments
Assignees

Comments

@brian-mann
Copy link
Member

@brian-mann brian-mann commented Jul 1, 2018

More comprehensive proposal is needed but summary is this:

  • Refactor existing debug / log implementation
  • Create logger wrapper utility to manage all logic
  • Route debug, console.log and performance data through this utility
  • Enable silencing all node based console.log per #2071
  • Capture browser console methods per #700
  • Write all logs to an output.log per project in AppData
  • Persist logs on recorded dashboard runs
  • Enable changing the logs folder via some CLI flag like --logs-folder=/path/to/logs
  • Fix morgan output always appearing
  • Expose a pluginEvent for internal:log perhaps
  • Start capturing performance related data in logs
@jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Dec 26, 2018

Would this issue cover this request? #2991

@oghmjos
Copy link

@oghmjos oghmjos commented Mar 15, 2019

As first step, having feature to upload text files from logs directory to dashboard would be very helpful and could be a good jumpstart. Will enhance cypress-failed-log plugin too with no extra effort. Maybe its already planned in the nearest future?

@jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Dec 17, 2020

Since this issue hasn't had activity in a while, we'll close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants