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

ASF GitHub Bot commented on DISPATCH-1646:
------------------------------------------

ChugR commented on a change in pull request #748:
URL: https://github.com/apache/qpid-dispatch/pull/748#discussion_r431247067



##########
File path: CMakeLists.txt
##########
@@ -61,8 +61,6 @@ message(STATUS "Found Proton: ${Proton_LIBRARIES} (found 
version \"${Proton_VERS
 
 ## Optional dependencies
 find_package(LibWebSockets)

Review comment:
       What if I want to build without LibWebSockets? There's no option any 
more.
   A good plan is to keep the option USE_LIBWEBSOCKETS and default it true 
before you try to find the package. Only look for the package if the option is 
true. Then if you find libwebsockets and the version is ok go ahead and use it.

##########
File path: CMakeLists.txt
##########
@@ -61,8 +61,6 @@ message(STATUS "Found Proton: ${Proton_LIBRARIES} (found 
version \"${Proton_VERS
 
 ## Optional dependencies
 find_package(LibWebSockets)

Review comment:
       Take a look at console/CMakeLists.txt and how it handles CONSOLE_INSTALL 
for a good pattern.




----------------------------------------------------------------
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:
[email protected]


> Unable to delete listener with http enabled
> -------------------------------------------
>
>                 Key: DISPATCH-1646
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-1646
>             Project: Qpid Dispatch
>          Issue Type: Improvement
>            Reporter: Ulf Lilleengen
>            Assignee: Ganesh Murthy
>            Priority: Major
>             Fix For: 1.13.0
>
>
> I'm running into an issue when trying to delete a listener which has 'http: 
> true' set. The router returns the error message "HTTP listeners cannot be 
> deleted". I can see that there is a test for this in the router code as well.
> However, in order for EnMasse to be able to create and delete listeners that 
> are used to handle websocket connections for different tenants, we need a way 
> to delete listeners with http: true set through AMQP management without 
> restarting the router.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to