The following document contains the results of FindBugs Report
FindBugs Version is 1.3.8
Threshold is Low
Effort is Default
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
461 | 14 | 0 | 0 |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should com.atlassian.gadgets.renderer.internal.AtlassianContainerConfig$Containers be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 26 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class com.atlassian.gadgets.renderer.internal.BlobCrypterImpl$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 39-51 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class com.atlassian.gadgets.renderer.internal.GadgetSpecFactoryImpl$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 83-121 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in com.atlassian.gadgets.renderer.internal.GadgetSpecUrlCheckerImpl.assertRenderable(String) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 89-92 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class com.atlassian.gadgets.renderer.internal.guice.ShindigModule$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 97 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class com.atlassian.gadgets.renderer.internal.guice.ShindigModule$2 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 108 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class com.atlassian.gadgets.renderer.internal.guice.ShindigModule$3 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 109 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 42 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
com.atlassian.gadgets.renderer.internal.servlet.AtlassianOAuthCallbackServlet is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 10-12 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
HTTP parameter written to Servlet error page in com.atlassian.gadgets.renderer.internal.servlet.GadgetSpecUrlRenderPermissionServletFilter.doFilter(ServletRequest, ServletResponse, FilterChain) | SECURITY | XSS_REQUEST_PARAMETER_TO_SEND_ERROR | 86 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
com.atlassian.gadgets.renderer.internal.servlet.RpcRelay is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 16-34 | Low |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class com.atlassian.gadgets.renderer.internal.servlet.SecurityTokenServlet defines non-transient non-serializable instance field decoder | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
Class com.atlassian.gadgets.renderer.internal.servlet.SecurityTokenServlet defines non-transient non-serializable instance field userManager | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
com.atlassian.gadgets.renderer.internal.servlet.SecurityTokenServlet is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31-86 | Low |