Skip to content
This repository has been archived by the owner on Jun 2, 2024. It is now read-only.

/package/sonido cause cpu 100% #552

Closed
fengmk2 opened this issue Jan 2, 2015 · 10 comments · Fixed by #553
Closed

/package/sonido cause cpu 100% #552

fengmk2 opened this issue Jan 2, 2015 · 10 comments · Fixed by #553
Assignees
@fengmk2
Labels
bug
Milestone
2.x

Comments

@fengmk2
Copy link
Member

fengmk2 commented Jan 2, 2015

image

207.46.13.133 - 10.159.63.226 [02/Jan/2015:16:10:22 +0800] "GET /package/sonido HTTP/1.0" 499 0 "-" "Mozilla/5.0 (compatible; bingbot/2.0; +http://www.bing.com/bingbot.htm)"
@fengmk2 fengmk2 added the bug label Jan 2, 2015
@fengmk2 fengmk2 self-assigned this Jan 2, 2015
@fengmk2 fengmk2 added this to the 2.0 milestone Jan 2, 2015
@fengmk2
Copy link
Member Author

fengmk2 commented Jan 2, 2015

cause by jonschlinkert/remarkable#127

@fengmk2 fengmk2 changed the title https://npm.taobao.org/package/sonido cause cpu 100% /package/sonido cause cpu 100% Jan 2, 2015
fengmk2 added a commit that referenced this issue Jan 2, 2015
@fengmk2
fix(markdown): use marked instead of remarkable
603fdee 
Fixes #552
@fengmk2 fengmk2 mentioned this issue Jan 2, 2015
Merged
@fengmk2
Copy link
Member Author

fengmk2 commented Jan 4, 2015

commonmark/commonmark-spec#263

@fengmk2 fengmk2 reopened this Jan 4, 2015
@fengmk2
Copy link
Member Author

fengmk2 commented Jan 4, 2015

marked also has cpu problem.

@puzrin
Copy link

puzrin commented Jan 4, 2015

All regexp-based parsers without clear state machine have tons of serious problems. CommonMark reference & markdown-it are much more safe. I've released new markdown-it. If existing docs are not enougth and you have problems with migration to 3.0.0 - kick me via tracker or gitter.

@fengmk2
Copy link
Member Author

fengmk2 commented Jan 4, 2015

@puzrin Thanks a lot, I will use markdown-it right now.

@fengmk2 fengmk2 closed this as completed in 38ed0a9 Jan 4, 2015
@fengmk2 fengmk2 mentioned this issue Jan 4, 2015
Closed
@fengmk2
Copy link
Member Author

fengmk2 commented Jan 4, 2015

spmjs/spmjs.io#89

@puzrin
Copy link

puzrin commented Jan 4, 2015

Does he need my help? It's a bit hard to understand via google translator.

I've left comment what to change in that commit for upgrade.

@fengmk2
Copy link
Member Author

fengmk2 commented Jan 4, 2015

@puzrin I had use markdown-it replace marked on 38ed0a9 and it worked. Thanks for your markdown-it@3.0.0.

@puzrin
Copy link

puzrin commented Jan 4, 2015

Ah, i thinked link to spmjs was for me.

No prob. You found a great bug in regexp, i'll be glad to help anytime.

@fengmk2
Copy link
Member Author

fengmk2 commented Jan 4, 2015

I ping you on npm website tracker.

@UbuntuEvangelist UbuntuEvangelist mentioned this issue Jun 21, 2023
Open
@thonatos thonatos mentioned this issue Jun 24, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@fengmk2 fengmk2

Labels
bug
Projects
None yet
Milestone
2.x
Development

Successfully merging a pull request may close this issue.

fix(markdown): use marked instead of remarkable cnpm/cnpmjs.org
2 participants
@fengmk2 @puzrin