Ananta Podder
Ananta Podder
@msbt I tried to replicate this bug but was unsuccessful. Please try updating Ghostfolio to the latest version.
@dtslvr Can I work on this issue?
@dtslvr I checked [Redactyl](https://github.com/rcjpisani/redactyl.js) and implemented it in our codebase. However, it does not allow `null` to be used as the redacted value; instead, it uses `"[REDACTED]"` as the default....
Hi @dtslvr, New implementation takes 50% of the original time to go through all test cases. I did not check with any benchmark. Please suggest me if we need to...
Ok I'll make changes and let you know.
I just checked the [implementation of redact()](https://github.com/rcjpisani/redactyl.js/blob/6d7752da4a4ab5746f4ceb8d982a2106b25800e2/src/index.js#L53) in redactyl.js, interestingly, it is the same approach I followed in #5073 . @dtslvr thoughts?
Hi @dtslvr, did you get a chance to review my changes?
@dtslvr I have attached link to two PR, 1. #5151 implements redactyle.js, whereas, 2. #5150 has optimized version of existing method. In my observation, #5150 has less complexity and tests...
@rajdip-b Would love to take this issue.
@rajdip-b I have reached to solution by implementing eslint based rule [id-match](https://eslint.org/docs/latest/rules/id-match). But as you mentioned in your requirement list, we need to validate secret and environment name too. If...