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.

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

Exception when parsing svn.log #763

Closed
Richargh opened this issue Oct 27, 2019 · 0 comments · Fixed by #766
Closed

Exception when parsing svn.log #763

Richargh opened this issue Oct 27, 2019 · 0 comments · Fixed by #766
Assignees
@alexhunziker
Labels
bug Only issues that describe bugs. pr-analysis Issues that touch the analysis pr(oject).

Comments

@Richargh
Copy link

Bug

Expected Behavior

I can run ccsh scmlogparser subversion.svn.log --input-format SVN_LOG -o subversion.svn.cc.json without an error.

Actual Behavior

I receive an java.lang.StringIndexOutOfBoundsException: String index out of range: 2

Screenshots

Screenshot 2019-10-27 at 10.52.59.png

Steps to Reproduce the Problem

  1. svn co https://svn.apache.org/repos/asf/subversion/trunk subversion
  2. svn log --verbose > ../subversion.svn.log
  3. ccsh scmlogparser subversion.svn.log --input-format SVN_LOG -o subversion.svn.cc.json

Specifications

  • Version: 1.37.0
  • Platform: macOS
@Richargh Richargh added the bug Only issues that describe bugs. label Oct 27, 2019
@Richargh Richargh changed the title StringIndexOutOfBoundsException Exception when parsing svn.log Oct 27, 2019
@alexhunziker alexhunziker added the pr-analysis Issues that touch the analysis pr(oject). label Oct 28, 2019
alexhunziker added a commit that referenced this issue Oct 28, 2019
@alexhunziker
increased minimum length for fileLine to avoid OutOfBounds Exception #…
3178c88 
…763
alexhunziker added a commit that referenced this issue Oct 28, 2019
@alexhunziker
better test for commit separator #763
0a5683e
alexhunziker added a commit that referenced this issue Oct 28, 2019
@alexhunziker
invalid commits do not are catched and ignored #763
0473905
alexhunziker added a commit that referenced this issue Oct 28, 2019
@alexhunziker
warn instead of throw exception if more than one modification fits fi…
31d420a 
…le name #763

(when can this happen? this happens if the commit message contains a line with a valid syntax for a modification in SVN logs. This is not a reason to abort therefore the change)
alexhunziker added a commit that referenced this issue Oct 28, 2019
@alexhunziker
update changelog #763
64e0029
@alexhunziker alexhunziker mentioned this issue Oct 28, 2019
Merged
11 tasks
@alexhunziker alexhunziker self-assigned this Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexhunziker alexhunziker

Labels
bug Only issues that describe bugs. pr-analysis Issues that touch the analysis pr(oject).
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bug/763/SVN parser more resilient MaibornWolff/codecharta
2 participants
@alexhunziker @Richargh