Skip to content

Commit

Permalink
update docs
Browse files Browse the repository at this point in the history
  • Loading branch information
badra001 committed Jun 17, 2021
1 parent 41b5cbd commit 6fff8aa
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 4 deletions.
11 changes: 9 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ and then you can continue with the rest of the operation. In Terraform 0.12, mo
## LDAP Settings
You will get a very weird error about the LDAP provider having an invalid DN format if you pass anything to `ldap_user`
variable that does not look like a DN (cn=something,...). Be sure you've got a valid username and password for
LDAP. It's less importatn if you do not have `enable_ldap_creation` set to true (not sure if it will fail though).
LDAP. It's less important if you do not have `enable_ldap_creation` set to true (not sure if it will fail though).

# Usage

Expand Down Expand Up @@ -73,6 +73,13 @@ module "myrole2" {

Creating a non-SAML role with inline policies
```hcl
data "aws_iam_document_policy" "my-policy-1" {
statement {
sid = "NameOfPermissiosn"
# rest of stuff
}
}
module "myrole3" {
source = "git@github.e.it.census.gov:terraform-modules/aws-iam-role.git"
Expand All @@ -81,7 +88,7 @@ module "myrole3" {
inline_policies = [
{
name = "my-policy-1"
policy = data.aws_iam_policy_documennt.my-policy-1.json
policy = data.aws_iam_policy_document.my-policy-1.json
}
]
}
Expand Down
11 changes: 9 additions & 2 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
* ## LDAP Settings
* You will get a very weird error about the LDAP provider having an invalid DN format if you pass anything to `ldap_user`
* variable that does not look like a DN (cn=something,...). Be sure you've got a valid username and password for
* LDAP. It's less importatn if you do not have `enable_ldap_creation` set to true (not sure if it will fail though).
* LDAP. It's less important if you do not have `enable_ldap_creation` set to true (not sure if it will fail though).
*
* # Usage
*
Expand Down Expand Up @@ -74,6 +74,13 @@
*
* Creating a non-SAML role with inline policies
* ```hcl
* data "aws_iam_document_policy" "my-policy-1" {
* statement {
* sid = "NameOfPermissiosn"
* # rest of stuff
* }
* }
*
* module "myrole3" {
* source = "git@github.e.it.census.gov:terraform-modules/aws-iam-role.git"
*
Expand All @@ -82,7 +89,7 @@
* inline_policies = [
* {
* name = "my-policy-1"
* policy = data.aws_iam_policy_documennt.my-policy-1.json
* policy = data.aws_iam_policy_document.my-policy-1.json
* }
* ]
* }
Expand Down

0 comments on commit 6fff8aa

Please sign in to comment.