VulnerableApp-facade icon indicating copy to clipboard operation
VulnerableApp-facade copied to clipboard

Published 20 hours ago • verified publisher icon SasanLabs

Add a github action to validate the PR.

Open preetkaran20 opened this issue 4 years ago • 3 comments

There is currently no Github action that validates the PR. Add the action to validate both javascript/typescript changes (npm run build) and also Lua changes in the PR.

preetkaran20 avatar Jul 05 '21 13:07 preetkaran20

Added the PR validation for the javascript/typescript module. Left part is LUA validation.

preetkaran20 avatar Sep 05 '21 11:09 preetkaran20

I can help could you provide further requirement details? thanks

lmcdo avatar Jun 06 '22 06:06 lmcdo

Hi @lmcdo ,

So currently, this project has TypeScript/Javascript as well as Lua file (for openresty) and we are looking for a github action to compile and validate the PR before merging it into master.

Currently the github actions is there at https://github.com/SasanLabs/VulnerableApp-facade/blob/main/.github/workflows/Compile.yml and it is working fine for Typescript/Javascript but we want to include the LUA validator as well. Also I am not sure, if we are running unit tests for Typescript/Javascript as part of the PR build so wanted to validate that as well.

LUA script is present at: https://github.com/SasanLabs/VulnerableApp-facade/tree/main/lua-modules. it is invoked from https://github.com/SasanLabs/VulnerableApp-facade/blob/main/nginx.conf (nginx.conf)

Please let me know, if you need more information.

thanks, Karan

preetkaran20 avatar Jun 06 '22 16:06 preetkaran20