[GitHub] [druid] ccaominh commented on pull request #10408: Web console: single instance of axios for all network requests

[GitBox <gi...@apache.org>]

ccaominh commented on pull request #10408:
URL: https://github.com/apache/druid/pull/10408#issuecomment-696405034


   > This PR currently breaks the existing configuration route.
   > Try adding `"customHeaders": { 'x-test-header': 'this-is-a-test' }` to the console-config.js file and see that the custom headers will not be applied. This is because while you are creating a custom instance you are still setting `Api.defaults` which will not be read now.
   > 
   > I suggest you make a method called `Api.initialize` that will take the configs and create the instance. It must be called before any API requests can go through (in `entry.ts`).
   
   Is it possible to add a test so that CI can help ensure this doesn't break in the future?


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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org