[ 
https://issues.apache.org/jira/browse/KNOX-2748?focusedWorklogId=840215&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-840215
 ]

ASF GitHub Bot logged work on KNOX-2748:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 19/Jan/23 10:42
            Start Date: 19/Jan/23 10:42
    Worklog Time Spent: 10m 
      Work Description: MrtnBalazs commented on code in PR #716:
URL: https://github.com/apache/knox/pull/716#discussion_r1081086392


##########
gateway-service-hashicorp-vault/src/test/java/org/apache/knox/gateway/backend/hashicorp/vault/TestHashicorpVaultAliasService.java:
##########
@@ -184,6 +185,9 @@ public void testVaultIntegration() throws Exception {
     assertNull(aliasService.getPasswordFromAliasForCluster(clusterName, 
alias));
     assertEquals(0, aliasService.getAliasesForCluster(clusterName).size());
 
+    char[] generatedPassword = 
aliasService.getPasswordFromAliasForCluster(clusterName, alias, true);
+    assertNotNull(generatedPassword != null);
+

Review Comment:
   Alright.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 840215)
    Time Spent: 0.5h  (was: 20m)

> HashicorpVaultAliasService getPasswordFromAliasForCluster looks incorrecct
> --------------------------------------------------------------------------
>
>                 Key: KNOX-2748
>                 URL: https://issues.apache.org/jira/browse/KNOX-2748
>             Project: Apache Knox
>          Issue Type: Bug
>            Reporter: Attila Magyar
>            Assignee: Marton Balázs
>            Priority: Major
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> {code}
>   @Override
>   public char[] getPasswordFromAliasForCluster(String clusterName, String 
> alias, boolean generate) throws AliasServiceException {
>     if(generate) {
>       getPasswordFromAliasForCluster(clusterName, alias);
>     }
>     return getPasswordFromAliasForCluster(clusterName, alias); // this is 
> just the same method as above
>   }
> {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to