[Bug 61337] Downgrade AssertionError to RecordFormatException?

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61337 Tim Allison changed: What|Removed |Added Summary|Downgrade AssertionError to |Downgrade

[Bug 61182] Invalid signature created for streamed xlsx file

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61182 --- Comment #6 from Andreas Beeker --- The windows/linux files differ in their line-endings, due to org.apache.xmlbeans.impl.store.Saver._newLine being system dependent. As the xml canonicalization handles the newlines

[Bug 61337] Downgrade AssertionError to RecordFormatException?

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61337 --- Comment #2 from Tim Allison --- Of course, there are many, many more without the paren: "assert " -- You are receiving this mail because: You are the assignee for the bug.

[Bug 61337] New: Downgrade AssertionError to RuntimeException or something better

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61337 Bug ID: 61337 Summary: Downgrade AssertionError to RuntimeException or something better Product: POI Version: 3.17-dev Hardware: PC Status: NEW

[Bug 61337] Downgrade AssertionError to RuntimeException or something better

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61337 Tim Allison changed: What|Removed |Added OS||All --- Comment #1

[Bug 61337] Downgrade AssertionError to RecordFormatException?

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61337 --- Comment #3 from Andreas Beeker --- +1 for replacing java-assert I think we need at least two different cases: asserts based on record format (throws RecordFormatException) and other asserts (throws ? extends

[Bug 61338] New: Avoid infinite loop in corrupt wmf

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61338 Bug ID: 61338 Summary: Avoid infinite loop in corrupt wmf Product: POI Version: 3.17-dev Hardware: PC Status: NEW Severity: normal Priority: P2

Re: [Bug 61182] Invalid signature created for streamed xlsx file

2017-07-25 Thread Greg Woolsey
Unless there is some use case I'm missing, unit tests would be the only place a newly written file hash would need to match a precalculated value, meaning c) should be fine by me. I don't think anyone should expect POI to read a file and have the saved result be binarily equal to the input. The

Build failed in Jenkins: POI-DSL-1.6 #433

2017-07-25 Thread Apache Jenkins Server
See -- [...truncated 16.25 KB...] at java.net.InetAddress.getByName(InetAddress.java:1076) at

[Bug 61337] Downgrade AssertionError to RecordFormatException?

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61337 --- Comment #4 from Tim Allison --- I'll probably start on this one class/family at a time unless there are objections. It looks like there are basically 4 places where we rely on assert. 1) assert that things are or

[Bug 61338] Avoid infinite loop in corrupt wmf

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61338 Tim Allison changed: What|Removed |Added Status|NEW |RESOLVED

Unsynchronize xmlbean calls

2017-07-25 Thread Andreas Beeker
Hi, does anyone mind, if we unsynchronize the xmlbean calls? [1] Andi [1] https://stackoverflow.com/questions/45082014/ signature.asc Description: OpenPGP digital signature

[Bug 61182] Invalid signature created for streamed xlsx file

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61182 --- Comment #7 from Andreas Beeker --- add hashes for other linebreaks via r1803011 -- You are receiving this mail because: You are the assignee for the bug.

[Bug 61343] Sheet.getFirstRowNum() returns first non empty row

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61343 Javen O'Neal changed: What|Removed |Added OS||All

[Bug 61343] Sheet.getFirstRowNum() returns first non empty row

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61343 --- Comment #2 from Javen O'Neal --- Refer to the Spreadsheet Quick Guide and examples for how to handle worksheets that may contain empty rows or cells. This is what you're looking for to return an empty row from a

[Bug 61343] New: Sheet.getFirstRowNum() returns first non empty row

2017-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=61343 Bug ID: 61343 Summary: Sheet.getFirstRowNum() returns first non empty row Product: POI Version: 3.16-FINAL Hardware: PC Status: NEW Severity: major Priority:

Build failed in Jenkins: POI-DSL-Maven #283

2017-07-25 Thread Apache Jenkins Server
See Changes: [kiwiwings] Bug 61182 - Invalid signature created for streamed xlsx file - add hashes for other linebreaks [tallison] 61338 -- avoid infinite loop triggered by fuzzed wmf file

Build failed in Jenkins: POI-DSL-OpenJDK #214

2017-07-25 Thread Apache Jenkins Server
See -- [...truncated 16.27 KB...] at java.net.InetAddress.getByName(InetAddress.java:1076) at

Re: Unsynchronize xmlbean calls

2017-07-25 Thread Greg Woolsey
+1, and good find by the SO user. On Tue, Jul 25, 2017, 15:31 Andreas Beeker wrote: > Hi, > > does anyone mind, if we unsynchronize the xmlbean calls? [1] > > Andi > > [1] https://stackoverflow.com/questions/45082014/ > >

[GUMP@vmgump-vm3]: Project poi (in module poi) failed

2017-07-25 Thread Gump [HEAD] :-(
To whom it may engage... This is an automated request, but not an unsolicited one. For more information please visit http://gump.apache.org/nagged.html, and/or contact the folk at gene...@gump.apache.org. Project poi has an issue affecting its community integration. This issue affects