Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(cli): Secret scan #367

Closed
wants to merge 21 commits into from
Closed

feat(cli): Secret scan #367

wants to merge 21 commits into from

Conversation

kriptonian1
Copy link
Contributor

Description

Give a summary of the change that you have made

Fixes #[ISSUENO]

Dependencies

Mention any dependencies/packages used

Future Improvements

Mention any improvements to be done in future related to any file/feature

Mentions

Mention and tag the people

Screenshots of relevant screens

Add screenshots of relevant screens

Developer's checklist

  • My PR follows the style guidelines of this project
  • I have performed a self-check on my work

If changes are made in the code:

  • I have followed the coding guidelines
  • My changes in code generate no new warnings
  • My changes are breaking another fix/feature of the project
  • I have added test cases to show that my feature works
  • I have added relevant screenshots in my PR
  • There are no UI/UX issues

Documentation Update

  • This PR requires an update to the documentation at docs.keyshade.xyz
  • I have made the necessary updates to the documentation, or no documentation changes are required.

@kriptonian1 kriptonian1 requested a review from rajdip-b July 26, 2024 08:29
Copy link
Contributor

Failed to generate code suggestions for PR

@kriptonian1 kriptonian1 changed the title Feat/cli scan Feat(cli): secret scan Jul 26, 2024
@rajdip-b rajdip-b changed the title Feat(cli): secret scan feat(cli): Secret scan Jul 26, 2024
packages/secret-scan/package.json Show resolved Hide resolved
apps/cli/package.json Outdated Show resolved Hide resolved
apps/cli/package.json Outdated Show resolved Hide resolved
apps/cli/src/commands/scan.command.ts Show resolved Hide resolved
apps/cli/test.txt Outdated Show resolved Hide resolved
apps/platform/Dockerfile Show resolved Hide resolved
packages/secret-scan/src/rules/jwt.ts Outdated Show resolved Hide resolved
@kriptonian1
Copy link
Contributor Author

I am gonna add the other secrets, and then we can merge it, still have to add another ~90 of them

@rajdip-b
Copy link
Member

I am gonna add the other secrets, and then we can merge it, still have to add another ~90 of them

Let's not bother too much about that at the moment and put it up for the community. We should get the platform ready.

@kriptonian1
Copy link
Contributor Author

I am gonna add the other secrets, and then we can merge it, still have to add another ~90 of them

Let's not bother too much about that at the moment and put it up for the community. We should get the platform ready.

I don't think that's a good idea

@kriptonian1
Copy link
Contributor Author

Because we are almost done here

@rajdip-b rajdip-b marked this pull request as draft July 29, 2024 13:36
@kriptonian1
Copy link
Contributor Author

@Sambit003 can you add the rest of the rules for the secrets

@Sambit003 Sambit003 self-assigned this Aug 17, 2024
@Sambit003
Copy link
Contributor

@Sambit003 can you add the rest of the rules for the secrets

already working on that

@Sambit003
Copy link
Contributor

@rajdip-b @kriptonian1 for an unknown reason, cloudflare is breaking other regex tests, so I've suppressed cloudflare across the files.
Should I take responsibility for opening a GH issue for these kind of things after merge??

@kriptonian1
Copy link
Contributor Author

@rajdip-b @kriptonian1 for an unknown reason, cloudflare is breaking other regex tests, so I've suppressed cloudflare across the files. Should I take responsibility for opening a GH issue for these kind of things after merge??

Sure man you own this code

@kriptonian1
Copy link
Contributor Author

@Sambit003 once you fix the conflicts, I will give it check and merge this

@Sambit003
Copy link
Contributor

@Sambit003 once you fix the conflicts, I will give it check and merge this

Yeah sure

Copy link
Contributor

codiumai-pr-agent-free bot commented Sep 16, 2024

CI Failure Feedback 🧐

(Checks updated until commit 59b596b)

Action: Validate Cli

Failed stage: Build [❌]

Relevant error logs:
1:  ##[group]Operating System
2:  Ubuntu
...

196:  .../node_modules/@swc/core postinstall: Done
197:  .../node_modules/@nestjs/core postinstall$ opencollective || exit 0
198:  .../node_modules/nice-napi install: Done
199:  .../node_modules/@sentry/cli postinstall: Done
200:  .../node_modules/@prisma/engines postinstall$ node scripts/postinstall.js
201:  .../node_modules/@sentry/profiling-node install: @sentry/profiling-node: Precompiled binary found, attempting to load /home/runner/work/keyshade/keyshade/node_modules/.pnpm/@[email protected]/node_modules/@sentry/profiling-node/lib/sentry_cpu_profiler-linux-x64-glibc-115.node
202:  .../node_modules/@sentry/profiling-node install: @sentry/profiling-node: Precompiled binary found, skipping build from source.
203:  .../node_modules/@sentry/profiling-node install: Done
204:  .../node_modules/secp256k1 install$ npm run rebuild || echo "Secp256k1 bindings compilation fail. Pure JS implementation will be used."
...

250:  .../node_modules/secp256k1 install: gyp info spawn args '-Goutput_dir=.'
251:  .../node_modules/secp256k1 install: gyp info spawn args ]
252:  .../node_modules/secp256k1 install: gyp info spawn make
253:  .../node_modules/secp256k1 install: gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
254:  .../node_modules/secp256k1 install: make: Entering directory '/home/runner/work/keyshade/keyshade/node_modules/.pnpm/[email protected]/node_modules/secp256k1/build'
255:  .../node_modules/secp256k1 install:   CXX(target) Release/obj.target/secp256k1/src/addon.o
256:  .../node_modules/secp256k1 install: In file included from ../../../../[email protected]/node_modules/nan/nan.h:174,
257:  .../node_modules/secp256k1 install:                  from ../src/addon.cc:2:
258:  .../node_modules/secp256k1 install: ../../../../[email protected]/node_modules/nan/nan_callbacks.h:55:23: error: ‘AccessorSignature’ is not a member of ‘v8’
259:  .../node_modules/secp256k1 install:    55 | typedef v8::Local<v8::AccessorSignature> Sig;
260:  .../node_modules/secp256k1 install:       |                       ^~~~~~~~~~~~~~~~~
261:  .../node_modules/secp256k1 install: ../../../../[email protected]/node_modules/nan/nan_callbacks.h:55:40: error: template argument 1 is invalid
262:  .../node_modules/secp256k1 install:    55 | typedef v8::Local<v8::AccessorSignature> Sig;
263:  .../node_modules/secp256k1 install:       |                                        ^
264:  .../node_modules/secp256k1 install: In file included from ../src/addon.cc:2:
265:  .../node_modules/secp256k1 install: ../../../../[email protected]/node_modules/nan/nan.h: In function ‘void Nan::AsyncQueueWorker(Nan::AsyncWorker*)’:
266:  .../node_modules/secp256k1 install: ../../../../[email protected]/node_modules/nan/nan.h:2298:7: warning: cast between incompatible function types from ‘void (*)(uv_work_t*)’ {aka ‘void (*)(uv_work_s*)’} to ‘uv_after_work_cb’ {aka ‘void (*)(uv_work_s*, int)’} [-Wcast-function-type]
267:  .../node_modules/secp256k1 install:  2298 |     , reinterpret_cast<uv_after_work_cb>(AsyncExecuteComplete)
268:  .../node_modules/secp256k1 install:       |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
269:  .../node_modules/secp256k1 install: ../../../../[email protected]/node_modules/nan/nan.h: In function ‘void Nan::SetAccessor(v8::Local<v8::ObjectTemplate>, v8::Local<v8::String>, Nan::GetterCallback, Nan::SetterCallback, v8::Local<v8::Value>, v8::AccessControl, v8::PropertyAttribute, Nan::imp::Sig)’:
270:  .../node_modules/secp256k1 install: ../../../../[email protected]/node_modules/nan/nan.h:2536:19: error: no matching function for call to ‘v8::ObjectTemplate::SetAccessor(v8::Local<v8::String>&, void (*&)(v8::Local<v8::Name>, const v8::PropertyCallbackInfo<v8::Value>&), void (*&)(v8::Local<v8::Name>, v8::Local<v8::Value>, const v8::PropertyCallbackInfo<void>&), v8::Local<v8::Object>&, v8::AccessControl&, v8::PropertyAttribute&, Nan::imp::Sig&)’
...

298:  .../node_modules/secp256k1 install:   816 |   void SetAccessor(
299:  .../node_modules/secp256k1 install:       |        ^~~~~~~~~~~
300:  .../node_modules/secp256k1 install: /home/runner/.cache/node-gyp/20.17.0/include/node/v8-template.h:821:22: note:   no known conversion for argument 7 from ‘Nan::imp::Sig’ {aka ‘int’} to ‘v8::SideEffectType’
301:  .../node_modules/secp256k1 install:   821 |       SideEffectType getter_side_effect_type = SideEffectType::kHasSideEffect,
302:  .../node_modules/secp256k1 install:       |       ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
303:  .../node_modules/secp256k1 install: In file included from ../../../../[email protected]/node_modules/nan/nan.h:2884,
304:  .../node_modules/secp256k1 install:                  from ../src/addon.cc:2:
305:  .../node_modules/secp256k1 install: ../../../../[email protected]/node_modules/nan/nan_typedarray_contents.h: In constructor ‘Nan::TypedArrayContents<T>::TypedArrayContents(v8::Local<v8::Value>)’:
306:  .../node_modules/secp256k1 install: ../../../../[email protected]/node_modules/nan/nan_typedarray_contents.h:34:43: error: ‘class v8::ArrayBuffer’ has no member named ‘GetContents’
...

312:  .../node_modules/secp256k1 install:  1232 |       (node::addon_register_func) (regfunc),                          \
313:  .../node_modules/secp256k1 install:       |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
314:  .../node_modules/secp256k1 install: /home/runner/.cache/node-gyp/20.17.0/include/node/node.h:1266:3: note: in expansion of macro ‘NODE_MODULE_X’
315:  .../node_modules/secp256k1 install:  1266 |   NODE_MODULE_X(modname, regfunc, NULL, 0)  // NOLINT (readability/null_usage)
316:  .../node_modules/secp256k1 install:       |   ^~~~~~~~~~~~~
317:  .../node_modules/secp256k1 install: ../src/addon.cc:50:1: note: in expansion of macro ‘NODE_MODULE’
318:  .../node_modules/secp256k1 install:    50 | NODE_MODULE(secp256k1, Init)
319:  .../node_modules/secp256k1 install:       | ^~~~~~~~~~~
320:  .../node_modules/secp256k1 install: make: *** [secp256k1.target.mk:160: Release/obj.target/secp256k1/src/addon.o] Error 1
321:  .../node_modules/secp256k1 install: make: Leaving directory '/home/runner/work/keyshade/keyshade/node_modules/.pnpm/[email protected]/node_modules/secp256k1/build'
322:  .../node_modules/secp256k1 install: gyp ERR! build error 
323:  .../node_modules/secp256k1 install: gyp ERR! stack Error: `make` failed with exit code: 2
324:  .../node_modules/secp256k1 install: gyp ERR! stack at ChildProcess.<anonymous> (/home/runner/work/keyshade/keyshade/node_modules/.pnpm/[email protected]/node_modules/npm/node_modules/node-gyp/lib/build.js:216:23)
325:  .../node_modules/secp256k1 install: gyp ERR! System Linux 6.5.0-1025-azure
326:  .../node_modules/secp256k1 install: gyp ERR! command "/opt/hostedtoolcache/node/20.17.0/x64/bin/node" "/home/runner/work/keyshade/keyshade/node_modules/.pnpm/[email protected]/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
327:  .../node_modules/secp256k1 install: gyp ERR! cwd /home/runner/work/keyshade/keyshade/node_modules/.pnpm/[email protected]/node_modules/secp256k1
328:  .../node_modules/secp256k1 install: gyp ERR! node -v v20.17.0
329:  .../node_modules/secp256k1 install: gyp ERR! node-gyp -v v10.2.0
330:  .../node_modules/secp256k1 install: gyp ERR! not ok 
331:  .../node_modules/secp256k1 install: Secp256k1 bindings compilation fail. Pure JS implementation will be used.
...

363:  .../[email protected]/node_modules/eccrypto install: gyp info spawn args '-Goutput_dir=.'
364:  .../[email protected]/node_modules/eccrypto install: gyp info spawn args ]
365:  .../[email protected]/node_modules/eccrypto install: gyp info spawn make
366:  .../[email protected]/node_modules/eccrypto install: gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
367:  .../[email protected]/node_modules/eccrypto install: make: Entering directory '/home/runner/work/keyshade/keyshade/node_modules/.pnpm/[email protected]/node_modules/eccrypto/build'
368:  .../[email protected]/node_modules/eccrypto install:   CXX(target) Release/obj.target/ecdh/ecdh.o
369:  .../[email protected]/node_modules/eccrypto install: In file included from ../../../../[email protected]/node_modules/nan/nan.h:174,
370:  .../[email protected]/node_modules/eccrypto install:                  from ../ecdh.cc:2:
371:  .../[email protected]/node_modules/eccrypto install: ../../../../[email protected]/node_modules/nan/nan_callbacks.h:55:23: error: ‘AccessorSignature’ is not a member of ‘v8’
372:  .../[email protected]/node_modules/eccrypto install:    55 | typedef v8::Local<v8::AccessorSignature> Sig;
373:  .../[email protected]/node_modules/eccrypto install:       |                       ^~~~~~~~~~~~~~~~~
374:  .../[email protected]/node_modules/eccrypto install: ../../../../[email protected]/node_modules/nan/nan_callbacks.h:55:40: error: template argument 1 is invalid
375:  .../[email protected]/node_modules/eccrypto install:    55 | typedef v8::Local<v8::AccessorSignature> Sig;
376:  .../[email protected]/node_modules/eccrypto install:       |                                        ^
377:  .../[email protected]/node_modules/eccrypto install: In file included from ../ecdh.cc:2:
378:  .../[email protected]/node_modules/eccrypto install: ../../../../[email protected]/node_modules/nan/nan.h: In function ‘void Nan::AsyncQueueWorker(Nan::AsyncWorker*)’:
379:  .../[email protected]/node_modules/eccrypto install: ../../../../[email protected]/node_modules/nan/nan.h:2298:7: warning: cast between incompatible function types from ‘void (*)(uv_work_t*)’ {aka ‘void (*)(uv_work_s*)’} to ‘uv_after_work_cb’ {aka ‘void (*)(uv_work_s*, int)’} [-Wcast-function-type]
380:  .../[email protected]/node_modules/eccrypto install:  2298 |     , reinterpret_cast<uv_after_work_cb>(AsyncExecuteComplete)
381:  .../[email protected]/node_modules/eccrypto install:       |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
382:  .../[email protected]/node_modules/eccrypto install: ../../../../[email protected]/node_modules/nan/nan.h: In function ‘void Nan::SetAccessor(v8::Local<v8::ObjectTemplate>, v8::Local<v8::String>, Nan::GetterCallback, Nan::SetterCallback, v8::Local<v8::Value>, v8::AccessControl, v8::PropertyAttribute, Nan::imp::Sig)’:
383:  .../[email protected]/node_modules/eccrypto install: ../../../../[email protected]/node_modules/nan/nan.h:2536:19: error: no matching function for call to ‘v8::ObjectTemplate::SetAccessor(v8::Local<v8::String>&, void (*&)(v8::Local<v8::Name>, const v8::PropertyCallbackInfo<v8::Value>&), void (*&)(v8::Local<v8::Name>, v8::Local<v8::Value>, const v8::PropertyCallbackInfo<void>&), v8::Local<v8::Object>&, v8::AccessControl&, v8::PropertyAttribute&, Nan::imp::Sig&)’
...

411:  .../[email protected]/node_modules/eccrypto install:   816 |   void SetAccessor(
412:  .../[email protected]/node_modules/eccrypto install:       |        ^~~~~~~~~~~
413:  .../[email protected]/node_modules/eccrypto install: /home/runner/.cache/node-gyp/20.17.0/include/node/v8-template.h:821:22: note:   no known conversion for argument 7 from ‘Nan::imp::Sig’ {aka ‘int’} to ‘v8::SideEffectType’
414:  .../[email protected]/node_modules/eccrypto install:   821 |       SideEffectType getter_side_effect_type = SideEffectType::kHasSideEffect,
415:  .../[email protected]/node_modules/eccrypto install:       |       ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
416:  .../[email protected]/node_modules/eccrypto install: In file included from ../../../../[email protected]/node_modules/nan/nan.h:2884,
417:  .../[email protected]/node_modules/eccrypto install:                  from ../ecdh.cc:2:
418:  .../[email protected]/node_modules/eccrypto install: ../../../../[email protected]/node_modules/nan/nan_typedarray_contents.h: In constructor ‘Nan::TypedArrayContents<T>::TypedArrayContents(v8::Local<v8::Value>)’:
419:  .../[email protected]/node_modules/eccrypto install: ../../../../[email protected]/node_modules/nan/nan_typedarray_contents.h:34:43: error: ‘class v8::ArrayBuffer’ has no member named ‘GetContents’
420:  .../[email protected]/node_modules/eccrypto install:    34 |       data   = static_cast<char*>(buffer->GetContents().Data()) + byte_offset;
421:  .../[email protected]/node_modules/eccrypto install:       |                                           ^~~~~~~~~~~
422:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc: In function ‘int derive(const uint8_t*, const uint8_t*, uint8_t*)’:
423:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:37:41: warning: ‘EC_KEY* EC_KEY_new_by_curve_name(int)’ is deprecated: Since OpenSSL 3.0 [-Wdeprecated-declarations]
424:  .../[email protected]/node_modules/eccrypto install:    37 |   CHECK((pkey = EC_KEY_new_by_curve_name(NID_secp256k1)) != NULL);
425:  .../[email protected]/node_modules/eccrypto install:       |                 ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~
426:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:14:32: note: in definition of macro ‘CHECK’
427:  .../[email protected]/node_modules/eccrypto install:    14 | #define CHECK(cond) do { if (!(cond)) goto error; } while (0)
...

429:  .../[email protected]/node_modules/eccrypto install: In file included from ../ecdh.cc:4:
430:  .../[email protected]/node_modules/eccrypto install: /home/runner/.cache/node-gyp/20.17.0/include/node/openssl/ec.h:998:31: note: declared here
431:  .../[email protected]/node_modules/eccrypto install:   998 | OSSL_DEPRECATEDIN_3_0 EC_KEY *EC_KEY_new_by_curve_name(int nid);
432:  .../[email protected]/node_modules/eccrypto install:       |                               ^~~~~~~~~~~~~~~~~~~~~~~~
433:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:38:31: warning: ‘int EC_KEY_set_private_key(EC_KEY*, const BIGNUM*)’ is deprecated: Since OpenSSL 3.0 [-Wdeprecated-declarations]
434:  .../[email protected]/node_modules/eccrypto install:    38 |   CHECK(EC_KEY_set_private_key(pkey, pkey_bn) == 1);
435:  .../[email protected]/node_modules/eccrypto install:       |         ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~
436:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:14:32: note: in definition of macro ‘CHECK’
437:  .../[email protected]/node_modules/eccrypto install:    14 | #define CHECK(cond) do { if (!(cond)) goto error; } while (0)
...

439:  .../[email protected]/node_modules/eccrypto install: In file included from ../ecdh.cc:4:
440:  .../[email protected]/node_modules/eccrypto install: /home/runner/.cache/node-gyp/20.17.0/include/node/openssl/ec.h:1056:27: note: declared here
441:  .../[email protected]/node_modules/eccrypto install:  1056 | OSSL_DEPRECATEDIN_3_0 int EC_KEY_set_private_key(EC_KEY *key, const BIGNUM *prv);
442:  .../[email protected]/node_modules/eccrypto install:       |                           ^~~~~~~~~~~~~~~~~~~~~~
443:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:40:29: warning: ‘int EVP_PKEY_set1_EC_KEY(EVP_PKEY*, ec_key_st*)’ is deprecated: Since OpenSSL 3.0 [-Wdeprecated-declarations]
444:  .../[email protected]/node_modules/eccrypto install:    40 |   CHECK(EVP_PKEY_set1_EC_KEY(evp_pkey, pkey) == 1);
445:  .../[email protected]/node_modules/eccrypto install:       |         ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
446:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:14:32: note: in definition of macro ‘CHECK’
447:  .../[email protected]/node_modules/eccrypto install:    14 | #define CHECK(cond) do { if (!(cond)) goto error; } while (0)
...

449:  .../[email protected]/node_modules/eccrypto install: In file included from ../ecdh.cc:3:
450:  .../[email protected]/node_modules/eccrypto install: /home/runner/.cache/node-gyp/20.17.0/include/node/openssl/evp.h:1372:5: note: declared here
451:  .../[email protected]/node_modules/eccrypto install:  1372 | int EVP_PKEY_set1_EC_KEY(EVP_PKEY *pkey, struct ec_key_st *key);
452:  .../[email protected]/node_modules/eccrypto install:       |     ^~~~~~~~~~~~~~~~~~~~
453:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:43:44: warning: ‘EC_KEY* EC_KEY_new_by_curve_name(int)’ is deprecated: Since OpenSSL 3.0 [-Wdeprecated-declarations]
454:  .../[email protected]/node_modules/eccrypto install:    43 |   CHECK((peerkey = EC_KEY_new_by_curve_name(NID_secp256k1)) != NULL);
455:  .../[email protected]/node_modules/eccrypto install:       |                    ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~
456:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:14:32: note: in definition of macro ‘CHECK’
457:  .../[email protected]/node_modules/eccrypto install:    14 | #define CHECK(cond) do { if (!(cond)) goto error; } while (0)
...

492:  .../[email protected]/node_modules/eccrypto install: In file included from ../ecdh.cc:4:
493:  .../[email protected]/node_modules/eccrypto install: /home/runner/.cache/node-gyp/20.17.0/include/node/openssl/ec.h:1122:27: note: declared here
494:  .../[email protected]/node_modules/eccrypto install:  1122 | OSSL_DEPRECATEDIN_3_0 int EC_KEY_set_public_key_affine_coordinates(EC_KEY *key,
495:  .../[email protected]/node_modules/eccrypto install:       |                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
496:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:69:29: warning: ‘int EVP_PKEY_set1_EC_KEY(EVP_PKEY*, ec_key_st*)’ is deprecated: Since OpenSSL 3.0 [-Wdeprecated-declarations]
497:  .../[email protected]/node_modules/eccrypto install:    69 |   CHECK(EVP_PKEY_set1_EC_KEY(evp_peerkey, peerkey) == 1);
498:  .../[email protected]/node_modules/eccrypto install:       |         ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~
499:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:14:32: note: in definition of macro ‘CHECK’
500:  .../[email protected]/node_modules/eccrypto install:    14 | #define CHECK(cond) do { if (!(cond)) goto error; } while (0)
...

523:  .../[email protected]/node_modules/eccrypto install:  1232 |       (node::addon_register_func) (regfunc),                          \
524:  .../[email protected]/node_modules/eccrypto install:       |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
525:  .../[email protected]/node_modules/eccrypto install: /home/runner/.cache/node-gyp/20.17.0/include/node/node.h:1266:3: note: in expansion of macro ‘NODE_MODULE_X’
526:  .../[email protected]/node_modules/eccrypto install:  1266 |   NODE_MODULE_X(modname, regfunc, NULL, 0)  // NOLINT (readability/null_usage)
527:  .../[email protected]/node_modules/eccrypto install:       |   ^~~~~~~~~~~~~
528:  .../[email protected]/node_modules/eccrypto install: ../ecdh.cc:131:1: note: in expansion of macro ‘NODE_MODULE’
529:  .../[email protected]/node_modules/eccrypto install:   131 | NODE_MODULE(ecdh, InitAll)
530:  .../[email protected]/node_modules/eccrypto install:       | ^~~~~~~~~~~
531:  .../[email protected]/node_modules/eccrypto install: make: *** [ecdh.target.mk:114: Release/obj.target/ecdh/ecdh.o] Error 1
532:  .../[email protected]/node_modules/eccrypto install: make: Leaving directory '/home/runner/work/keyshade/keyshade/node_modules/.pnpm/[email protected]/node_modules/eccrypto/build'
533:  .../[email protected]/node_modules/eccrypto install: gyp ERR! build error 
534:  .../[email protected]/node_modules/eccrypto install: gyp ERR! stack Error: `make` failed with exit code: 2
...

652:  �[32mCJS�[39m �[1mdist/index.js �[22m�[32m46.31 KB�[39m
653:  �[32mCJS�[39m ⚡️ Build success in 193ms
654:  �[34mDTS�[39m Build start
655:  �[32mDTS�[39m ⚡️ Build success in 1312ms
656:  �[32mDTS�[39m �[1mdist/index.d.ts �[22m�[32m356.00 B�[39m
657:  ##[endgroup]
658:  �[;31mcli:build�[;0m
659:  cache miss, executing 8ec31574e4a63cf2
660:  ##[error]cli#build: command (/home/runner/work/keyshade/keyshade/apps/cli) /home/runner/setup-pnpm/node_modules/.bin/pnpm run build exited (1)
...

666:  �[34mCLI�[39m Using tsup config: /home/runner/work/keyshade/keyshade/apps/cli/tsup.config.ts
667:  �[34mCLI�[39m Target: es2022
668:  �[34mCLI�[39m Cleaning output folder
669:  �[34mCJS�[39m Build start
670:  "readdir" is imported from external module "fs/promises" but never used in "dist/index.js".
671:  �[34mDTS�[39m Build start
672:  �[32mCJS�[39m �[1mdist/index.js �[22m�[32m321.41 KB�[39m
673:  �[32mCJS�[39m ⚡️ Build success in 781ms
674:  error TS5074: Option '--incremental' can only be specified using tsconfig, emitting to single file or when option '--tsBuildInfoFile' is specified.
675:  �[31mError: error occured in dts build
676:  at Worker.<anonymous> (/home/runner/work/keyshade/keyshade/node_modules/.pnpm/[email protected]_@[email protected][email protected][email protected][email protected][email protected][email protected]/node_modules/tsup/dist/index.js:1539:26)
677:  at Worker.emit (node:events:519:28)
678:  at MessagePort.<anonymous> (node:internal/worker:263:53)
679:  at [nodejs.internal.kHybridDispatch] (node:internal/event_target:820:20)
680:  at MessagePort.<anonymous> (node:internal/per_context/messageport:23:28)�[39m
681:  �[31mDTS�[39m �[31mBuild error�[39m
682:  ELIFECYCLE  Command failed with exit code 1.
683:  [ERROR] command finished with error: command (/home/runner/work/keyshade/keyshade/apps/cli) /home/runner/setup-pnpm/node_modules/.bin/pnpm run build exited (1)
684:  Tasks:    1 successful, 2 total
685:  Cached:    0 cached, 2 total
686:  Time:    6.917s 
687:  Failed:    cli#build
688:  ERROR  run failed: command  exited (1)
689:  ELIFECYCLE  Command failed with exit code 1.
690:  ##[error]Process completed with exit code 1.

✨ CI feedback usage guide:

The CI feedback tool (/checks) automatically triggers when a PR has a failed check.
The tool analyzes the failed checks and provides several feedbacks:

  • Failed stage
  • Failed test name
  • Failure summary
  • Relevant error logs

In addition to being automatically triggered, the tool can also be invoked manually by commenting on a PR:

/checks "https://github.com/{repo_name}/actions/runs/{run_number}/job/{job_number}"

where {repo_name} is the name of the repository, {run_number} is the run number of the failed check, and {job_number} is the job number of the failed check.

Configuration options

  • enable_auto_checks_feedback - if set to true, the tool will automatically provide feedback when a check is failed. Default is true.
  • excluded_checks_list - a list of checks to exclude from the feedback, for example: ["check1", "check2"]. Default is an empty list.
  • enable_help_text - if set to true, the tool will provide a help message with the feedback. Default is true.
  • persistent_comment - if set to true, the tool will overwrite a previous checks comment with the new feedback. Default is true.
  • final_update_message - if persistent_comment is true and updating a previous checks message, the tool will also create a new message: "Persistent checks updated to latest commit". Default is true.

See more information about the checks tool in the docs.

@Sambit003 Sambit003 requested a review from rajdip-b September 16, 2024 18:03
@rajdip-b rajdip-b closed this Sep 19, 2024
@kriptonian1
Copy link
Contributor Author

@rajdip-b why did you closed it ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants