search

DannyDainton / newman-reporter-htmlextra

A HTML reporter for Postman's Command Line Runner, Newman. Includes Non Aggregated Runs broken down by Iterations, Skipped Tests, Console Logs and the handlebars helpers module for better custom templates.
Apache License 2.0
356 stars 137 forks source link

Log pollution and job failures created by the colors.js dependency #350

Closed floppyrom closed 2 years ago

floppyrom commented 2 years ago

I am using the latest version of the reporter (installed with npm install -g newman-reporter-htmlextra).

When I run my newman builds with the htmlextra reporter I get this error and all my builds are failing. I think it's related to the colors.js issue (https://www.bleepingcomputer.com/news/security/dev-corrupts-npm-libs-colors-and-faker-breaking-thousands-of-apps/)

and I get this in the logs of my builds: LIBERTY LIBERTY LIBERTY LIBERTY LIBERTY LIBERTY LIBERTY LIBERTY LIBERTY ! H|H|H|H|H H__ H|��|��|��|H H| |---------------------| H|��|���|��|H H| |---------------------| H|��|��|��|H H| |---------------------| H|H|H|H|H H| |---------------------| H|H|H|H|H H|---------------------------------| =============== H|---------------------------------| /| | H|---------------------------------| (| O O |) H|---------------------------------| /| U | H----------------------------------- | =/ | H .../ H |I/| H /| H |/ H / / / H | | | / | H | ||o|| | H | | ||o|| | | H | | ||o|| | | H Carl Pilcher
t��������������������������������������e����������������������������������������������������������������������������������������������s��������������������������������������t������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������i������������������������������������������������������n����������������������������������������������������������������������������������������������������������������������������������������������������������g���������������������������������� ���������������������������������������������������� ���������������������������������� ����������������������t��������������������������������������������������������������������������������������������������������e������������������������������������������������������������������������������������������������������������������������s����������������������������������������������������������������������������������������������������������������������t������������������

DannyDainton commented 2 years ago

My bad, I did make that change on the repo last week but didn't publish a new version 😢

I've published 1.22.4 with the changes, you're probably going to need to do a full uninstall/install of the reporter or potentially drop the node_modules and install again.

Completely my fault, apologies for any inconvenience this has caused.

floppyrom commented 2 years ago

No worries and thank you very much for the fix! :)

DannyDainton commented 2 years ago

Let me know if that's solved your issue, if not we can work together to get you going again.

floppyrom commented 2 years ago

Hi Danny

Apologies for the late reply (I had some other technical issues which prevented me to rerun my tests). It's working right now, so I can confirm your fix. Thanks again.

DannyDainton commented 2 years ago

Glad it's all sorted. Closing this out now.

github-actions[bot] commented 2 years ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.