KuthumiPepple commented on PR #3227: URL: https://github.com/apache/camel-k/pull/3227#issuecomment-1111097981
> I had approved already, but I had a look at this last commit [b573c5b](https://github.com/apache/camel-k/commit/b573c5bfa90c57190f3594f447b1b8c515b93629) after the approval and I am not sure why we want to expose the whole template. Can you please clarify the goal behind this last change? Thanks. The goal is to make modifications (both replacements and additions) to the template easier. Like in this case I no longer have to retrieve, replace and reset (cobra sets it the first time) the template string just to make a modification. All I had to do was modify the needed area in the template. Without exposing the template, future alterations (especially additions) to it would be cumbersome. I also couldn't think of any security concerns not to do so but I may be wrong. -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
