- Issue created by @bisonbleu
- Merge request !720Update ToolsExplorer::getResponse() to handle redirects. → (Open) created by Unnamed author
- 🇩🇪Germany marcus_johansson
That logic should imho not be in the tool explorer or be a solution that actions solves - and if the url action doesn't work well, we should remove it and create an actual function call for it.
We can't have conditionals for specific actions, outside of the ActionPluginDeriver, and even there we should have very little logic. Its also unclear if the goto action actually would work together with an agent, since the trigger is an API request. We should rather build a way for function calls to interact via JS.
So, I will close this, because the function call actions is not the solution to this problem from practical point of view, but also security point of view. I'm sorry about you did put work into this and it did not make it into the code, but this would just be a small band-aid trying to solve a huge gash.
- 🇨🇦Canada bisonbleu
NP, c'est la vie. Thank you @Marcus for taking the time to explain. You expose intricate & implicit details that I could only partially fathom. Now I better understand. Moving on… : )