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

Markdown.ToHtml throws ArgumentOutOfRangeException #275

Closed
Metalnem opened this Issue Dec 24, 2018 · 3 comments

Comments

2 participants
@Metalnem
Copy link

Metalnem commented Dec 24, 2018

Markdown.ToHtml throws an ArgumentOutOfRangeException when converting the markdown file from the attached archive. You can run the following code to reproduce it (the path variable should contain the path to the extracted file):

var text = File.ReadAllText(path);
var pipeline = new MarkdownPipelineBuilder().UseAdvancedExtensions().Build();
Markdown.ToHtml(text, pipeline);

Found via SharpFuzz.

@xoofx

This comment has been minimized.

Copy link
Member

xoofx commented Dec 24, 2018

Please stop reporting with an automated tool. I don't have time to download an archive and check the issues. So I prefer to have a simple code snippet with a reproducible error. Thank you

@xoofx xoofx closed this Dec 24, 2018

xoofx added a commit that referenced this issue Dec 28, 2018

@xoofx xoofx added the bug label Dec 28, 2018

@xoofx

This comment has been minimized.

Copy link
Member

xoofx commented Dec 28, 2018

The issue should be fixed by commit 03858dc

I would appreciate if next time you can include the full exception directly in your report (to give more context).

Thanks for the fuzz testing. my apologize if I was angry when receiving these notifications but that was really a bad timing for me getting bombarded with issues while preparing Christmas diner 😅

The fact that the report looks like an automated stuffs was really scary for me at first, I thought I would receive hundred of them in the following minutes, hence blocking you straight.

@Metalnem

This comment has been minimized.

Copy link

Metalnem commented Dec 28, 2018

My bad about the missing stack trace! I started including them in reports for the other repositories the next day 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment