Skip to content
This repository has been archived by the owner on Mar 31, 2019. It is now read-only.

javax.crypto.NoSuchPaddingException exception thrown randomly #25

Closed
kovax opened this issue May 13, 2018 · 0 comments
Closed

javax.crypto.NoSuchPaddingException exception thrown randomly #25

kovax opened this issue May 13, 2018 · 0 comments
Assignees
@kovax
Labels
bug critical
Milestone
3.4

Comments

@kovax
Copy link
Member

kovax commented May 13, 2018 •
edited
Loading

This must be a concurrency issue decrypting the cookie string.

EXCEPTION:javax.crypto.BadPaddingException: Given final block not properly padded
	at com.sun.crypto.provider.CipherCore.doFinal(CipherCore.java:989)
	at com.sun.crypto.provider.CipherCore.doFinal(CipherCore.java:845)
	at com.sun.crypto.provider.AESCipher.engineDoFinal(AESCipher.java:446)
	at javax.crypto.Cipher.doFinal(Cipher.java:2165)
	at org.cristalise.restapi.RestHandler.decryptAuthData(RestHandler.java:111)
	at org.cristalise.restapi.RestHandler.checkAuthData(RestHandler.java:156)
	at org.cristalise.restapi.RestHandler.checkAuthCookie(RestHandler.java:140)
	at org.cristalise.restapi.ItemRoot.getItemSummary(ItemRoot.java:209)
	at sun.reflect.GeneratedMethodAccessor101.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
@kovax kovax self-assigned this May 13, 2018
kovax added a commit that referenced this issue May 13, 2018
@kovax
#25 - Add logging to capture the issue
3e2287c
kovax added a commit that referenced this issue May 15, 2018
@kovax
#25 -remove extra logging. it was not usefull
15bb700
kovax added a commit that referenced this issue May 19, 2018
@kovax
#25 - all read/write access to cookie is synchronized
0c46add
kovax added a commit that referenced this issue May 20, 2018
@kovax
#25 - update jersey/grizzly/jackson dependencies to a recent version
8182060
@taurus227 taurus227 mentioned this issue May 22, 2018
Merged
@kovax kovax closed this as completed in #29 May 22, 2018
kovax added a commit that referenced this issue May 22, 2018
@kovax
Merge pull request #29 from cristal-ise/Unauthorized401
79f0f0f 
TitusTech/webui-primeng#191: 401_Unauthorized problem: In case of BadPadd… closes #25
@kovax kovax added this to the 3.4 milestone Jun 24, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees

@kovax kovax

Labels
bug critical
Projects
None yet
Milestone
3.4
Development

No branches or pull requests
1 participant
@kovax