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

[JsSrc2Cpg] - Handle runtime exception gracefully when filtering astGen result #5281

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

khemrajrathore
Copy link
Contributor

No description provided.

@max-leuthaeuser
Copy link
Contributor

How did an invalid path end up there? What do you mean by "invalid"?
Could you provide any test case?

@khemrajrathore khemrajrathore changed the title [JsSrc2Cpg] - Handle invalid file path when reading astGen result [JsSrc2Cpg] - Handle runtime exception gracefully when reading astGen result Feb 3, 2025
@khemrajrathore
Copy link
Contributor Author

My bad, the PR title was incorrect. This PR is to avoid running into any runtime exception if any and instead handle the exception gracefully. Have updated the PR

@max-leuthaeuser max-leuthaeuser changed the title [JsSrc2Cpg] - Handle runtime exception gracefully when reading astGen result [JsSrc2Cpg] - Handle runtime exception gracefully when filtering astGen result Feb 3, 2025
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.

2 participants