[ 
https://issues.apache.org/jira/browse/RANGER-1442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15904132#comment-15904132
 ] 

Jeffrey E  Rodriguez edited comment on RANGER-1442 at 3/10/17 12:41 AM:
------------------------------------------------------------------------

Hi Yu Jie,
     I wonder if instead of treating DEFAULT_SHUTDOWN_PORT as a variable which 
may be incorrect from code style point of view. If we want to set the 
shutdownPort to SSL port you could change (in the 2 places where 
DEFAULT_SHUTDOWN is used) as : 
        int shutdownPort = getIntConfig("ranger.service.shutdown.port", 
""ranger.service.https.port"" )
If we do that when we may want to  get rid of DEFAULT_SHUTDOWN_PORT constant.






was (Author: jeffreyr97):
Hi Yu Jie,
     I wonder if instead of treating DEFAULT_SHUTDOWN_PORT as a variable which 
may be incorrect from code style point of view. If we want to set the 
shutdownPort to SSL port you could change (in the 2 places where 
DEFAULT_SHUTDOWN is defined) as : 
        int shutdownPort = getIntConfig("ranger.service.shutdown.port", 
""ranger.service.https.port"" )
If we do that when we may want to  get rid of DEFAULT_SHUTDOWN_PORT constant.





> https port not configurable
> ---------------------------
>
>                 Key: RANGER-1442
>                 URL: https://issues.apache.org/jira/browse/RANGER-1442
>             Project: Ranger
>          Issue Type: Bug
>          Components: admin
>    Affects Versions: 0.6.2
>            Reporter: Yujie Li
>             Fix For: 0.6.2
>
>         Attachments: 0001-RANGER-1442-https-port-not-configurable.patch
>
>
> Ranger doesn't take the https port value (default 6182) from 
> ranger-admin-site.xml and uses hardcoded value (6185) after enabling SSL



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to