You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tinkerpop.apache.org by GitBox <gi...@apache.org> on 2020/12/05 16:36:43 UTC

[GitHub] [tinkerpop] vtslab commented on pull request #1308: TINKERPOP-2389 Authorization support in TinkerPop

vtslab commented on pull request #1308:
URL: https://github.com/apache/tinkerpop/pull/1308#issuecomment-739318320


   At this moment, I have no loose ends left, so I call this the end of the present iteration. I want to mention one particular choice: I stopped considering adding authorization for the WsAndHttpChannelizer for the following reasons:
   
   - the WsAndHttpChannelizer is an almost undocumented feature
   - its implementation is already kind of a hack
   - authorization depends on the Authenticated user, which is provided independently on the http and ws channels and might even be different
   
   Of course, it does not mean that it could not be added later on.


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