Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix(workaround): RangeError: Invalid string length #123

Merged
merged 2 commits into from
Apr 13, 2022
Merged

fix(workaround): RangeError: Invalid string length #123

merged 2 commits into from
Apr 13, 2022

Conversation

yoshinorin
Copy link
Member

What is this?

It the user has a many & huge posts. The JSON.stringify throw the RangeError. The error caused by V8 constraint.

refs: hexojs/hexo#4922

Workaround

catch the exception type of RangeError before save to db.json.
A side-effect of this workaround is the db.json will not generate.

Others

I assume currently, we can't deal with anything about this issue (Stream are may effective but I did not try to implement it.) .

@yoshinorin yoshinorin mentioned this pull request Apr 12, 2022
Closed
@yoshinorin yoshinorin merged commit b5a3ba7 into hexojs:master Apr 13, 2022
@yoshinorin yoshinorin deleted the fix/hexo-invalid-string-length branch April 13, 2022 12:29
@yoshinorin yoshinorin mentioned this pull request Apr 13, 2022
Merged
@Starryi Starryi mentioned this pull request May 17, 2022
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@SukkaW SukkaW SukkaW approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yoshinorin @SukkaW