projectdiscovery/nuclei-action
1
0
Fork 0
Code Issues Pull requests 6 Projects Releases 10 Packages Wiki Activity Actions

Infrequent error: "Cannot read properties of undefined (reading 'replace')" #51

New issue
Closed
opened 2023-01-13 09:28:32 -05:00 by allanbreyes · 3 comments
allanbreyes commented 2023-01-13 09:28:32 -05:00 (Migrated from github.com)
Copy link

Hi, team! We infrequently receive an error, maybe ~1-5% of the time:

Error: Cannot read properties of undefined (reading 'replace')

There's only 3 places in the codebase that call .replace, and I suspect it's the call to getLatestInfo sometimes having a transient error. I think it probably needs a retry, or at least some slightly better error-handling and logging. I can take a stab at it if I have some free cycles next week. Thoughts?

Hi, team! We infrequently receive an error, maybe ~1-5% of the time: > Error: Cannot read properties of undefined (reading 'replace') There's only 3 places in the codebase that call `.replace`, and I suspect it's the call to [getLatestInfo](https://github.com/projectdiscovery/nuclei-action/blob/6fdcee9529d5da992b2e9a4b599ee7688d145ed2/src/installer.js#L39) sometimes having a transient error. I think it probably needs a retry, or at least some slightly better error-handling and logging. I can take a stab at it if I have some free cycles next week. Thoughts?
👍 2
CCDi commented 2024-07-12 14:34:37 -04:00 (Migrated from github.com)
Copy link

Hi team! Any thoughts/updates on this error? We bumped into it once again! 🙏

Hi team! Any thoughts/updates on this error? We bumped into it once again! 🙏
RamanaReddy0M commented 2024-07-24 12:47:18 -04:00 (Migrated from github.com)
Copy link

@allanbreyes @CCDi added undefinded/existence checks before applying replace(..)

Error: Cannot read properties of undefined (reading 'replace')

will be fixed with https://github.com/projectdiscovery/nuclei-action/pull/88

@allanbreyes @CCDi added undefinded/existence checks before applying `replace(..)` > Error: Cannot read properties of undefined (reading 'replace') will be fixed with https://github.com/projectdiscovery/nuclei-action/pull/88
🎉 2
CCDi commented 2024-11-25 18:23:34 -05:00 (Migrated from github.com)
Copy link

Hi ya'll, thank you so much for fixing this issue! Would you mind releasing a new version of the nuclei-action so we can utilize this fix?

Hi ya'll, thank you so much for fixing this issue! Would you mind releasing a new version of the nuclei-action so we can utilize this fix?
👍 1
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
dependabot/npm_and_yarn/actions/core-2.0.1
dependabot/npm_and_yarn/actions/exec-2.0.0
dependabot/npm_and_yarn/js-yaml-4.1.1
dependabot/npm_and_yarn/vercel/ncc-0.38.4
dependabot/npm_and_yarn/actions/tool-cache-2.0.2
v2.0.2
v2.0.1
v2.0.0
v1.0.7
v1.0.6
v1.0.5
v1.0.4
v1.0.3
v1.0.1
v1.0.2
v1.0.0
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
projectdiscovery/nuclei-action#51
No description provided.