You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@knox.apache.org by GitBox <gi...@apache.org> on 2020/03/11 20:55:30 UTC

[GitHub] [knox] smolnar82 opened a new pull request #288: KNOX-2289 - Passing GatewayServer.getGatewayServices() to SimpleDescriptorHandler.handle() as it needs it to provision encryption query string password

smolnar82 opened a new pull request #288: KNOX-2289 - Passing GatewayServer.getGatewayServices() to SimpleDescriptorHandler.handle() as it needs it to provision encryption query string password
URL: https://github.com/apache/knox/pull/288
 
 
   ## What changes were proposed in this pull request?
   
   (Please fill in changes proposed in this fix)
   
   ## How was this patch tested?
   
   Manually tested:
   
   - redeployed Knox with my changes
   - had a CM descriptor with two topologies: `topology1` and `topology2`
   - started Knox: the CM descriptor got picked up and generated into toplogies
   - confirmed that:
     - the previous ERROR message - `Failed to create a password for query string encryption for` - did not show up in the log
     - the password was created
   
   ```
   $ bin/knoxcli.sh list-alias --cluster topology2
   Listing aliases for: topology2
   encryptquerystring
   
   1 items.
   
   $ bin/knoxcli.sh list-alias --cluster topology1
   Listing aliases for: topology1
   encryptquerystring
   
   1 items.
   ```

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


With regards,
Apache Git Services

[GitHub] [knox] smolnar82 merged pull request #288: KNOX-2289 - Passing GatewayServer.getGatewayServices() to SimpleDescriptorHandler.handle() as it needs it to provision encryption query string password

Posted by GitBox <gi...@apache.org>.
smolnar82 merged pull request #288: KNOX-2289 - Passing GatewayServer.getGatewayServices() to SimpleDescriptorHandler.handle() as it needs it to provision encryption query string password
URL: https://github.com/apache/knox/pull/288
 
 
   

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


With regards,
Apache Git Services

[GitHub] [knox] smolnar82 commented on issue #288: KNOX-2289 - Passing GatewayServer.getGatewayServices() to SimpleDescriptorHandler.handle() as it needs it to provision encryption query string password

Posted by GitBox <gi...@apache.org>.
smolnar82 commented on issue #288: KNOX-2289 - Passing GatewayServer.getGatewayServices() to SimpleDescriptorHandler.handle() as it needs it to provision encryption query string password
URL: https://github.com/apache/knox/pull/288#issuecomment-598215936
 
 
   I'd to rebase locally and submit a new PS due to changes in #287 (classes were re-factored out from DefaultTopologyService).

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


With regards,
Apache Git Services

[GitHub] [knox] risdenk commented on issue #288: KNOX-2289 - Passing GatewayServer.getGatewayServices() to SimpleDescriptorHandler.handle() as it needs it to provision encryption query string password

Posted by GitBox <gi...@apache.org>.
risdenk commented on issue #288: KNOX-2289 - Passing GatewayServer.getGatewayServices() to SimpleDescriptorHandler.handle() as it needs it to provision encryption query string password
URL: https://github.com/apache/knox/pull/288#issuecomment-597877824
 
 
   Ha that explains something that I don't have to track down now:
   
   ```
   2020-03-03 21:17:42,936 ERROR topology.simple (SimpleDescriptorHandler.java:handle(176)) - Failed to create a password for query string encryption for cdp-proxy-api.
   ```

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


With regards,
Apache Git Services