Skip to content

feat:Limit SQL length to avoid Stack Overflow #223

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 14 commits into from
Jun 14, 2024
Merged

feat:Limit SQL length to avoid Stack Overflow #223

merged 14 commits into from
Jun 14, 2024

Conversation

yanni97
Copy link
Contributor

@yanni97 yanni97 commented Jun 6, 2024 •
edited
Loading

  1. The catch exception type is changed to Throwable
  2. Limit SQL length to avoid Stack Overflow
  3. The maximum length of sql is read from the configuration
  4. Replace the [ and ] symbols

@yanni97 yanni97 changed the title Feat/catch type feat:Limit SQL length to avoid Stack Overflow Jun 7, 2024
@arextest arextest deleted a comment from QizhengMo Jun 12, 2024
coryhh
coryhh previously approved these changes Jun 13, 2024
View reviewed changes
coryhh
coryhh previously approved these changes Jun 14, 2024
View reviewed changes
mr3
mr3 previously approved these changes Jun 14, 2024
View reviewed changes
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@coryhh coryhh merged commit 2bce9e9 into arextest:main Jun 14, 2024
3 checks passed
@yanni97 yanni97 deleted the feat/catchType branch July 4, 2024 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lucas-myx lucas-myx lucas-myx left review comments

@pangdayuan1 pangdayuan1 pangdayuan1 left review comments

@mr3 mr3 mr3 approved these changes

@coryhh coryhh coryhh 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.
5 participants
@yanni97 @mr3 @lucas-myx @coryhh @pangdayuan1