FindBugs Bug Detector Report
The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is
Effort is max
Summary
| Classes |
Bugs |
Errors |
Missing Classes |
| 32 |
18 |
0 |
0 |
fr.paris.lutece.plugins.identitystore.v1.web.service.HttpAccessTransport
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.identitystore.v1.web.service.HttpAccessTransport.doDelete(String, Map, Map, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
192 |
Medium |
| fr.paris.lutece.plugins.identitystore.v1.web.service.HttpAccessTransport.doGet(String, Map, Map, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
127 |
Medium |
| fr.paris.lutece.plugins.identitystore.v1.web.service.HttpAccessTransport.doPostMultiPart(String, Map, Map, Map, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
160 |
Medium |
fr.paris.lutece.plugins.identitystore.v2.web.rs.service.MockIdentityTransportRest
| Bug |
Category |
Details |
Line |
Priority |
| Unread field: fr.paris.lutece.plugins.identitystore.v2.web.rs.service.MockIdentityTransportRest.CONNECTION_ID_PREFIX; should this field be static? |
PERFORMANCE |
SS_SHOULD_BE_STATIC |
57 |
Medium |
| Unread field: fr.paris.lutece.plugins.identitystore.v2.web.rs.service.MockIdentityTransportRest.CUSTOMER_ID_PREFIX; should this field be static? |
PERFORMANCE |
SS_SHOULD_BE_STATIC |
58 |
Medium |
fr.paris.lutece.plugins.identitystore.v2.web.service.HttpAccessTransport
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.identitystore.v2.web.service.HttpAccessTransport.doDelete(String, Map, Map, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
225 |
Medium |
| fr.paris.lutece.plugins.identitystore.v2.web.service.HttpAccessTransport.doGet(String, Map, Map, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
160 |
Medium |
| fr.paris.lutece.plugins.identitystore.v2.web.service.HttpAccessTransport.doPostMultiPart(String, Map, Map, Map, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
193 |
Medium |
fr.paris.lutece.plugins.identitystore.v3.web.rs.service.AbstractTransportRest
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.identitystore.v3.web.rs.service.AbstractTransportRest._mapper isn't final but should be |
MALICIOUS_CODE |
MS_SHOULD_BE_FINAL |
53 |
High |
fr.paris.lutece.plugins.identitystore.v3.web.rs.service.HttpAccessTransport
| Bug |
Category |
Details |
Line |
Priority |
| Exception is caught when Exception is not thrown in fr.paris.lutece.plugins.identitystore.v3.web.rs.service.HttpAccessTransport.mapJsonList(ObjectMapper, String, Class) |
STYLE |
REC_CATCH_EXCEPTION |
421 |
Medium |
| fr.paris.lutece.plugins.identitystore.v3.web.rs.service.HttpAccessTransport.doDelete(String, Map, Map, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
236 |
Medium |
| fr.paris.lutece.plugins.identitystore.v3.web.rs.service.HttpAccessTransport.doDeleteJSON(String, Map, Map, Object, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
272 |
Medium |
| fr.paris.lutece.plugins.identitystore.v3.web.rs.service.HttpAccessTransport.doGet(String, Map, Map, Class, ObjectMapper) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
204 |
Medium |
fr.paris.lutece.plugins.identitystore.v3.web.rs.service.MockIdentityTransportDataStore
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.identitystore.v3.web.rs.service.MockIdentityTransportDataStore._mapper isn't final but should be |
MALICIOUS_CODE |
MS_SHOULD_BE_FINAL |
74 |
High |
fr.paris.lutece.plugins.identitystore.v3.web.rs.service.MockIdentityTransportRest
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.identitystore.v3.web.rs.service.MockIdentityTransportRest._mapper isn't final but should be |
MALICIOUS_CODE |
MS_SHOULD_BE_FINAL |
82 |
High |
| Unread field: fr.paris.lutece.plugins.identitystore.v3.web.rs.service.MockIdentityTransportRest.CONNECTION_ID_PREFIX; should this field be static? |
PERFORMANCE |
SS_SHOULD_BE_STATIC |
75 |
Medium |
| Unread field: fr.paris.lutece.plugins.identitystore.v3.web.rs.service.MockIdentityTransportRest.CUSTOMER_ID_PREFIX; should this field be static? |
PERFORMANCE |
SS_SHOULD_BE_STATIC |
76 |
Medium |
fr.paris.lutece.plugins.identitystore.v3.web.rs.service.MockServiceContractTransportDataStore
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.identitystore.v3.web.rs.service.MockServiceContractTransportDataStore._mapper isn't final but should be |
MALICIOUS_CODE |
MS_SHOULD_BE_FINAL |
60 |
High |
