You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by "Gautam Borad (JIRA)" <ji...@apache.org> on 2015/11/19 12:25:11 UTC

[jira] [Updated] (RANGER-733) Implement best coding practices to resolve issues found during code scan

     [ https://issues.apache.org/jira/browse/RANGER-733?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gautam Borad updated RANGER-733:
--------------------------------
    Fix Version/s: 0.5.1

> Implement best coding practices to resolve issues found during code scan
> ------------------------------------------------------------------------
>
>                 Key: RANGER-733
>                 URL: https://issues.apache.org/jira/browse/RANGER-733
>             Project: Ranger
>          Issue Type: Bug
>    Affects Versions: 0.5.0, 0.6.0
>            Reporter: Velmurugan Periasamy
>            Assignee: Gautam Borad
>             Fix For: 0.5.1, 0.6.0
>
>
> *+Fix reported issues in below classes:+*
> +File:  org/apache/ranger/json/JsonDateSerializer.java+
> Call to method of static java.text.DateFormat in org.apache.ranger.json.JsonDateSerializer.serialize(Date, JsonGenerator, SerializerProvider)
> +Method: org.apache.ranger.biz.XUserMgr.updateXModuleDefPermission(VXModuleDef)+
> Make sure that  getIsAllowed()  is done without AUTOBOXING
> +File:  org/apache/ranger/service/XAccessAuditService.java+
> Field XAccessAuditService.searchFields masks field in superclass org.apache.ranger.service.AbstractBaseResourceService
> Field XAccessAuditService.sortFields masks field in superclass org.apache.ranger.service.AbstractBaseResourceService
> +File:  org/apache/ranger/service/XPolicyService.java+
> Line#:  239
> if (assetType == 0 || assetType == AppConstants.ASSET_UNKNOWN) -  are same condition!
> +File:  org/apache/ranger/view/VXGroupPermission.java+ 
> Field VXGroupPermission.id masks field in superclass org.apache.ranger.view.VXDataObject
> +File:  org/apache/ranger/view/VXModuleDef.java+
> Field VXModuleDef.id masks field in superclass org.apache.ranger.view.VXDataObject
> +File:  org/apache/ranger/view/VXUserPermission.java+
> Field VXUserPermission.id masks field in superclass org.apache.ranger.view.VXDataObject
> *+Fix logging in below classes:+*
> ./unixauthclient/src/main/java/org/apache/ranger/authentication/unix/jaas/RemoteUnixLoginModule.java
> ./lookup-client/src/main/java/org/apache/ranger/storm/client/StormClient.java



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)