TypeError: logger.info is not a function about ssl HOT 6 CLOSED

Looking into this now, seems we missed the logging methods in the last release for the base library. Will update with tests to cover as well. Fix will need to happen in the base library over at http://github.com/passmarked/passmarked which all the workers are based on. Will update as I go, thanks for reporting !

from ssl.

No problem, would have put in a PR for this myself but I kind of figured that the fix might have been in the base library as you said, didn't want to break convention by changing one or the other without knowing.

from ssl.

See passmarked/passmarked#19, doing some initial testing and will do a release to npm after I can confirm that your sample runs on this side too

from ssl.

Righto, tested and working nicely this side with your sample. Pushed through a release with a patch bump. You're looking to upgrade to 0.5.5 from https://www.npmjs.com/package/passmarked.

Give it a shot and let me know if you have more luck, after which I'll close the issue if all is well.

Just a note on the code sample provided for the SSL rules you'll be safer starting a single runner that you execute with, IE:

let passmarked = require('passmarked'),
    ssl = require('@passmarked/ssl');

let runner = passmarked.createRunner(ssl);
runner.run({url: 'testagent.cgos.info'})
    .then(info => console.log(info))
    .catch(err => console.log(err.stack));

from ssl.

I see what you mean, my script was a simple test intended to just run once then exit but definitely for reuse in an application I would follow that approach as well.

from ssl.

Closing to mark as done, feel free to report if anything else blocks the way :)

from ssl.