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

spacegrep error when parsing a markdown file #2987

Closed
minusworld opened this issue Apr 22, 2021 · 2 comments · Fixed by #3418
Closed

spacegrep error when parsing a markdown file #2987

minusworld opened this issue Apr 22, 2021 · 2 comments · Fixed by #3418
Assignees
Labels
blocked cannot proceed further at the moment bug Something isn't working lang:generic generic mode issues (spacegrep, aliengrep) priority:low user:external requested by someone outside of r2c

Comments

@minusworld
Copy link
Member

Describe the bug
spacegrep encounters an error when parsing the markdown file in issue #2985.

To Reproduce

  1. ❌ This is the original issue with a reduced pattern for easier debugging. This fails: https://semgrep.dev/s/ljGo/
  2. ✅ Removing a big chunk of the markdown file works: https://semgrep.dev/s/YgJd/
  3. ❌ Removing each line until it fails, this is the point at which it fails: https://semgrep.dev/s/qrWO/
  4. ❌ Removing a big chunk from the top half does not work: https://semgrep.dev/s/6rPK/
  5. ✅ Extremely minimal file, only two lines: https://semgrep.dev/s/ok3X/
  6. ❌ Minimal version with the "offending" line from (3) above: https://semgrep.dev/s/zkZn/

My hypothesis include:

  1. Might be some kind of timeout behavior, and certain lines take a while to parse.
  2. Might be related to backticks, which are in abundance and I haven't seen used extensively in spacegrep documents

What is the priority of the bug to you?
P2

@minusworld
Copy link
Member Author

Also, I tested this locally on the CLI (version 0.48.0) and it fails for the listed examples

@colleend colleend added bug Something isn't working lang:generic generic mode issues (spacegrep, aliengrep) priority:low user:external requested by someone outside of r2c labels Apr 22, 2021
@YakDriver
Copy link

Our team is anxiously awaiting this! Thanks for your work on it.

@nbrahms nbrahms added the blocked cannot proceed further at the moment label Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked cannot proceed further at the moment bug Something isn't working lang:generic generic mode issues (spacegrep, aliengrep) priority:low user:external requested by someone outside of r2c
Development

Successfully merging a pull request may close this issue.

5 participants