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

ASF GitHub Bot commented on AIRAVATA-2279:
------------------------------------------

machristie commented on a change in pull request #75: [AIRAVATA-2279 & 2280] 
FIXED.
URL: 
https://github.com/apache/airavata-php-gateway/pull/75#discussion_r148894369
 
 

 ##########
 File path: app/controllers/AdminController.php
 ##########
 @@ -421,33 +434,36 @@ public function getExperimentsOfTimeRange()
     }
 
        public function createSSH(){
-               $newToken = AdminUtilities::create_ssh_token_for_gateway(null);
+        $description = Input::get("description");
+        if($description == "Default SSH Key"){
 
 Review comment:
   I don't think this check is necessary.
   
   On the other hand, it might be nice to prevent admins from creating two keys 
with the same description.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Gateway credential store UI also to have the same format as individual user 
> settings
> ------------------------------------------------------------------------------------
>
>                 Key: AIRAVATA-2279
>                 URL: https://issues.apache.org/jira/browse/AIRAVATA-2279
>             Project: Airavata
>          Issue Type: Bug
>          Components: PGA PHP Web Gateway
>         Environment: dev.seagrid.org
>            Reporter: Eroma
>            Assignee: Marcus Christie
>            Priority: Major
>             Fix For: 0.18
>
>
> Currently there are no description ora copy button to copy the public key in 
> gateway credential store.
> Unifying the UI with User settings will make it easy for the gateway admins.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to