Skip to content

fix: 3657 #3657 - #3658

Merged
wenshao merged 4 commits into
alibaba:mainfrom
mengnankkkk:fix-3657
Jul 27, 2025
Merged

fix: 3657 #3657#3658
wenshao merged 4 commits into
alibaba:mainfrom
mengnankkkk:fix-3657

Conversation

@mengnankkkk

@mengnankkkk mengnankkkk commented Jul 23, 2025

Copy link
Copy Markdown
Contributor

What this PR does / why we need it?

Found that the maxLevel limit in the JSONWriter class is hard-coded to 2048
Context uses defaultMaxLevel when initialized but there is no public API to modify it

Summary of your change

Added two new public methods in JSONFactory.java

Please indicate you've done the following:

  • Made sure tests are passing and test coverage is added if needed.
  • Made sure commit message follow the rule of Conventional Commits specification.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@mengnankkkk mengnankkkk changed the title fix: 3657 fix: 3657 #3657 Jul 23, 2025
@wenshao

wenshao commented Jul 23, 2025

Copy link
Copy Markdown
Member

你需要在本地运行mvn validate来修复checkstyle的问题

@wenshao
wenshao merged commit 09c40a6 into alibaba:main Jul 27, 2025
13 checks passed
@daovjnhs2

Copy link
Copy Markdown

版本:fastjson2.0.62
我想要去配置maxlevel,我在fastjson2.properties中配置了maxlevel,但是这里的读取逻辑有问题,无法读取到我properties中的配置
image

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.

3 participants