jr-instantsystem commented on PR #13025:
URL: https://github.com/apache/apisix/pull/13025#issuecomment-4067728342

   Hi @moonming,
   
   I don't know how to cache the resolved conf after `fetch_secrets(conf)`, 
tried to look on other plugins but I don't get how to do it... But, they all do 
`fetch_secrets()` in `_M.access` or  `_M.rewrite` as you said, so I just moved 
the line there.
   
   + this fix an issue linked to my change for version 3.13+ (I'm running on 
3.12)
   
   + I had to implement elasticsearch datastream support for our internal 
needs, it's pushed on this PR too...
   Could be moved on an other ticket/PR if required, or we change the desc on 
this issue/PR, I don't know, you tell me :)
   


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

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to