Closed ShahanaFarooqui closed 3 months ago
[!WARNING]
Review failed
The pull request is closed.
This update focuses on enhancing the robustness of the removeAuthSecureData
method in the CommonService
class across various files by adding a check for node.authentication
before attempting to delete properties. Additionally, the project version has been incremented from "0.15.1-beta" to "0.15.2-beta" in relevant configuration files.
Files | Change Summary |
---|---|
backend/utils/common.js |
Modified CommonService 's removeAuthSecureData to check for node.authentication . |
server/utils/common.ts |
Updated CommonService 's removeAuthSecureData to include a check for node.authentication . |
package.json , src/.../consts-enums-functions.ts |
Version updated from "0.15.1-beta" to "0.15.2-beta". |
In the code where data weave resides,
A check for auth now safely abides. 🛡️
Robustness blooms where bugs once tread,
Secure and strong, the data's thread. 🌻
Version bumped, a step so neat,
Another milestone we did meet. 🎉🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
Bug Fixes
Chores
0.15.1-beta
to0.15.2-beta
.