[PR] Bump com.github.spotbugs from 5.2.1 to 6.0.10 [poi]

2024-04-09 Thread via GitHub


dependabot[bot] opened a new pull request, #619:
URL: https://github.com/apache/poi/pull/619

   Bumps com.github.spotbugs from 5.2.1 to 6.0.10.
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=com.github.spotbugs=gradle=5.2.1=6.0.10)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot show  ignore conditions` will show all of 
the ignore conditions of the specified dependency
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org



Re: [PR] fix for bug63189 [poi]

2024-04-09 Thread via GitHub


pjfanning commented on PR #617:
URL: https://github.com/apache/poi/pull/617#issuecomment-2044964656

   I think the reason the test might have appeared to work is because the test 
in the PR has a catch block that justs prints the stack trace but then swallows 
the exception. We cannot accept tests that swallow exceptions.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org



Build failed in Jenkins: POI ยป POI-DSL-SonarQube-Gradle #880

2024-04-09 Thread Apache Jenkins Server
See 


Changes:

[PJ Fanning] bouncycastle 1.77

[PJ Fanning] bouncycastle 1.77


--
[...truncated 2.41 MB...]
M P UrF: Unread field: org.apache.poi.hmef.HMEFMessage.fileId  At 
HMEFMessage.java:[line 69]
M V MS: org.apache.poi.hslf.usermodel.HSLFSimpleShape.ADJUST_VALUES should be 
package protected  At HSLFSimpleShape.java:[line 65]
M V MS: org.apache.poi.hwmf.usermodel.HwmfPicture.MAX_RECORD_LENGTH should be 
package protected  At HwmfPicture.java:[line 59]
SpotBugs ended with exit code 1

> Task :poi:spotbugsMain
M D NP: Load of known null value in 
org.apache.poi.ss.formula.FormulaParser.parseStructuredReference(String)  At 
FormulaParser.java:[line 839]
M D SF: Switch statement found in 
org.apache.poi.ss.formula.FormulaParser.parseSheetName(boolean) where default 
case is missing  At FormulaParser.java:[lines 1203-1213]
M B PI: The used identifier FormulaParser.java:[lines 1490-1519] as method name 
in class org.apache.poi.ss.formula.FormulaParser.Arguments() at line ?>?3/3??? 
as a method name shadows the publicly available identifier from the Java 
Standard Library.  At FormulaParser.java:[lines 1490-1519]
M D RCN: Redundant nullcheck of is which is known to be null in 
org.apache.poi.sl.draw.DrawPaint.getTexturePaint(PaintStyle$TexturePaint, 
Graphics2D)  Redundant null check at DrawPaint.java:[line 342]
M D NP: Load of known null value in 
org.apache.poi.sl.draw.DrawPaint.getTexturePaint(PaintStyle$TexturePaint, 
Graphics2D)  At DrawPaint.java:[line 342]
M V EI2: new org.apache.poi.sl.draw.DrawPaint(PlaceableShape) may expose 
internal representation by storing an externally mutable object into 
DrawPaint.shape  At DrawPaint.java:[line 81]
M D SF: Switch statement found in 
org.apache.poi.sl.draw.DrawPaint.getTexturePaint(PaintStyle$TexturePaint, 
Graphics2D) where default case is missing  At DrawPaint.java:[lines 301-310]
M V EI2: new org.apache.poi.sl.draw.DrawTextParagraph(TextParagraph) may expose 
internal representation by storing an externally mutable object into 
DrawTextParagraph.paragraph  At DrawTextParagraph.java:[line 113]
M V EI: org.apache.poi.ss.formula.OperationEvaluationContext.getWorkbook() may 
expose internal representation by returning 
OperationEvaluationContext._workbook  At OperationEvaluationContext.java:[line 
89]
M V EI2: new 
org.apache.poi.ss.formula.OperationEvaluationContext(WorkbookEvaluator, 
EvaluationWorkbook, int, int, int, EvaluationTracker, boolean) may expose 
internal representation by storing an externally mutable object into 
OperationEvaluationContext._workbook  At OperationEvaluationContext.java:[line 
72]
M V EI2: new 
org.apache.poi.ss.formula.OperationEvaluationContext(WorkbookEvaluator, 
EvaluationWorkbook, int, int, int, EvaluationTracker, boolean) may expose 
internal representation by storing an externally mutable object into 
OperationEvaluationContext._bookEvaluator  At 
OperationEvaluationContext.java:[line 71]
M V EI: org.apache.poi.poifs.crypt.agile.EncryptionDocument.getKeyData() may 
expose internal representation by returning EncryptionDocument.keyData  At 
EncryptionDocument.java:[line 98]
M V EI2: 
org.apache.poi.poifs.crypt.agile.EncryptionDocument.setDataIntegrity(DataIntegrity)
 may expose internal representation by storing an externally mutable object 
into EncryptionDocument.dataIntegrity  At EncryptionDocument.java:[line 110]
M V EI2: 
org.apache.poi.poifs.crypt.agile.EncryptionDocument.setKeyData(KeyData) may 
expose internal representation by storing an externally mutable object into 
EncryptionDocument.keyData  At EncryptionDocument.java:[line 102]
M V EI: org.apache.poi.poifs.crypt.agile.EncryptionDocument.getKeyEncryptors() 
may expose internal representation by returning 
EncryptionDocument.keyEncryptors  At EncryptionDocument.java:[line 114]
M V EI: org.apache.poi.poifs.crypt.agile.EncryptionDocument.getDataIntegrity() 
may expose internal representation by returning 
EncryptionDocument.dataIntegrity  At EncryptionDocument.java:[line 106]
M V EI2: 
org.apache.poi.poifs.crypt.agile.CertificateKeyEncryptor.setEncryptedKeyValue(byte[])
 may expose internal representation by storing an externally mutable object 
into CertificateKeyEncryptor.encryptedKeyValue  At 
CertificateKeyEncryptor.java:[line 76]
M V EI: 
org.apache.poi.poifs.crypt.agile.CertificateKeyEncryptor.getEncryptedKeyValue() 
may expose internal representation by returning 
CertificateKeyEncryptor.encryptedKeyValue  At 
CertificateKeyEncryptor.java:[line 72]
M V EI: 
org.apache.poi.poifs.crypt.agile.CertificateKeyEncryptor.getX509Certificate() 
may expose internal representation by returning 
CertificateKeyEncryptor.x509Certificate  At CertificateKeyEncryptor.java:[line 
80]
M V EI2: 
org.apache.poi.poifs.crypt.agile.CertificateKeyEncryptor.setCertVerifier(byte[])
 may expose internal representation by storing an