search

mcdcorp / opentest

Open source test automation tool for web applications, mobile apps and APIs
https://getopentest.org
MIT License
446 stars 107 forks source link

Disallow printing the encryptionPassword property or OPENTEST_ENCRYPTION_PASSWORD env variable #646

Closed ewmartinez closed 1 week ago

ewmartinez commented 3 weeks ago

Hi, Now it is posible to print the encryptionPassword or OPENTEST_ENCRYPTION_PASSWORD variable. It will be goot to mask that string, to prevent printing to log that value I see that it has to be done on getEncryptor methot, I want to ask via pull request the modification to add that functionality

pull https://github.com/mcdcorp/opentest/pull/647

Thanks, Edgar

adrianth commented 2 weeks ago

The pull request was merged and this change is now available is the 1.4.2 release that was published in beta. If you want to try it out, you can install it with

npm install opentest@beta -g

If you see any problems with the 1.4.2 release please report them in a new issue. If you need to revert to the current stable version you can always do that with

npm install opentest -g
ewmartinez commented 2 weeks ago

Hi @adrianth It is working fine thanks For me the ticket is ready to close, but I don't know if you want to realease first the new versión?

I take this opportunity to ask about when the new version would be available more or less and to learn about the new features that come with it.

Thanks Edgar

adrianth commented 1 week ago

Hi @ewmartinez, 1.4.2 only contains security updates and three small updates: your pull request; another pull request that adds the ability to filter test session templates list in the UI (useful if you have a lot of templates in your test repo); a small bug fix.

I'll wait a while before promoting 1.4.2 as "latest" because I want to allow time for early adopters to start using it and report potential issues.

Please close this issue if the functionality you're after works as expected.

ewmartinez commented 1 week ago

Hi @adrianth Thanks..