You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2020/06/29 17:21:36 UTC

[GitHub] [activemq-artemis] vkamble60 opened a new pull request #3206: ARTEMIS-2826 SSL configuration for 'data imp' tool

vkamble60 opened a new pull request #3206:
URL: https://github.com/apache/activemq-artemis/pull/3206


   Jira https://issues.apache.org/jira/browse/ARTEMIS-2826 


----------------------------------------------------------------
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



[GitHub] [activemq-artemis] jbertram edited a comment on pull request #3206: ARTEMIS-2826 SSL configuration for 'data imp' tool

Posted by GitBox <gi...@apache.org>.
jbertram edited a comment on pull request #3206:
URL: https://github.com/apache/activemq-artemis/pull/3206#issuecomment-651267827


   Instead of adding a handful of new input parameters to deal with SSL/TLS specifics I recommend deprecating the `host` and `port` parameters and adding a `url` parameter. That way *any* parameter can be set if desired (including SSL/TLS stuff).


----------------------------------------------------------------
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



[GitHub] [activemq-artemis] vkamble60 commented on pull request #3206: ARTEMIS-2826 SSL configuration for 'data imp' tool

Posted by GitBox <gi...@apache.org>.
vkamble60 commented on pull request #3206:
URL: https://github.com/apache/activemq-artemis/pull/3206#issuecomment-651660454


   @jbertram Thank you for recommendation. I was thinking same approach.. let me try with  URL parameter.


----------------------------------------------------------------
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



[GitHub] [activemq-artemis] jbertram closed pull request #3206: ARTEMIS-2826 SSL configuration for 'data imp' tool

Posted by GitBox <gi...@apache.org>.
jbertram closed pull request #3206:
URL: https://github.com/apache/activemq-artemis/pull/3206


   


----------------------------------------------------------------
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



[GitHub] [activemq-artemis] jbertram commented on pull request #3206: ARTEMIS-2826 SSL configuration for 'data imp' tool

Posted by GitBox <gi...@apache.org>.
jbertram commented on pull request #3206:
URL: https://github.com/apache/activemq-artemis/pull/3206#issuecomment-670039534


   Closing this for now. Re-open when you've got a solution using the URL.


----------------------------------------------------------------
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



[GitHub] [activemq-artemis] jbertram commented on pull request #3206: ARTEMIS-2826 SSL configuration for 'data imp' tool

Posted by GitBox <gi...@apache.org>.
jbertram commented on pull request #3206:
URL: https://github.com/apache/activemq-artemis/pull/3206#issuecomment-651267827


   Instead of adding a handful of new input parameters to deal with SSL/TLS specifics I recommend deprecating the `host` and `port` parameters and adding a `url` parameter. That way *any* parameter can be set if desired (include SSL/TLS stuff).


----------------------------------------------------------------
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