-
Notifications
You must be signed in to change notification settings - Fork 131
Conversation
🎊 PR Preview has been successfully built and deployed to https://localstack-docs-preview-pr-1363.surge.sh 🎊 |
5e80d26
to
5bde26f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approved it, so you can merge it right afterwards, but would love to have my comments addressed before merging (they are not big ones though).
Most importantly, we should use either ENFORCE_IAM
alone (then the requests will be blocked), or just IAM_SOFT_MODE
(you do not need ENFORCE_IAM
for this one anymore), to only show the logs in LS, but do not actively block request. Either would be fine for this tutorial, but we should correctly describe it :)
I added a banner image to match the other tutorials, if you wanna merge it |
https://localstack-docs-preview-pr-1363.surge.sh/tutorials/iam-policy-stream/