stoty commented on PR #912:
URL: https://github.com/apache/knox/pull/912#issuecomment-2107383158

   I have followed up on this, and this won't work with some service 
definitions.
   
   A few service definition, like hbaseui 2.1.0 have a template in the metadata 
context field.
   AFAICT this never gets used anywhere, as up until now context was just an 
informative field, but this means that my current patch is guaranteed not not 
work with those services.
   
   Either 
   * We need to change the context field for those context values for a 
non-templated one
   * We need to find/define another field to use for the cookie context
   * We need to determine the context some other way like calculating a prefix 
of all route entries.


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

To unsubscribe, e-mail: dev-unsubscr...@knox.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to