Skip to content
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

Batch-3745 #3843

Closed
wants to merge 2 commits into from
Closed

Batch-3745 #3843

wants to merge 2 commits into from

Conversation

nfjcore
Copy link
Contributor

@nfjcore nfjcore commented Feb 2, 2021

bug fix for Batch-3745.

Method public void write(String str, int off, int len) of org.springframework.batch.support.transaction.TransactionAwareBufferedWriter now calls StringBuilder.append(CharSequence, int, int) correct when off > len.

sync fork with spring-batch master
@pivotal-issuemaster
Copy link

@nfjcore Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@nfjcore Thank you for signing the Contributor License Agreement!

@timtebeek
Copy link

Fixes #3745

@ChristophKreibich
Copy link

I had an issue with an IndexOutOfBounds error while marshalling xml as describe here: https://stackoverflow.com/questions/66476051/jaxb2marshaller-problem-with-string-that-contains

I downloaded the source for 4.3.x, added the patch from this pull request, build and tested successfully with my application. This pull request is solving the issue.

@fmbenhassine
Copy link
Contributor

LGTM. Rebased and merged as 63b6dea. Thank you for your contribution!

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

Successfully merging this pull request may close these issues.

Incorrect usage of StringBuilder in TransactionAwareBufferedWriter
5 participants