jfrazee commented on pull request #5119: URL: https://github.com/apache/nifi/pull/5119#issuecomment-854069557
@turcsanyip On paper this LGTM. I'll try to give it a test. This ties into the discussion in #4843 I think, right? So we're walking back the change just for SAS tokens. I think that makes sense and is a good compromise because the use case is a solid one and SAS tokens present much less risk than the access key. Would it make sense to add more detail to `additionalDetails.html` on the trade offs and maybe suggestions around paying attention to queue and provenance policies? -- 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: [email protected]
