Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Converted Assume Role to C# and also Fixed Secret Key Access to Non-Plain Text #744

Merged
merged 16 commits into from
Jul 15, 2020

Conversation

zephyrz73
Copy link
Contributor

  1. Conversion assume-role to cs
  2. Encryped secret access key
  3. Indicate encryption in read me
  4. Add tests for aws-cs-assume-role and delete test for aws serverless-raw

@zephyrz73 zephyrz73 changed the title Zephyrz73/assume role conversion cs Converted Assume Role to C# and also Fixed Secret Key Access to Non-Plain Test Jul 8, 2020
@zephyrz73 zephyrz73 changed the title Converted Assume Role to C# and also Fixed Secret Key Access to Non-Plain Test Converted Assume Role to C# and also Fixed Secret Key Access to Non-Plain Text Jul 8, 2020
Copy link
Member

@justinvp justinvp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a 67.6 MB binary aws-go-assume-role/assume-role/assume-role that should be deleted.

Some minor other things, but looks good otherwise!

Note: There are links in the root README.md to the AssumeRole example for TypeScript and Python. Want to add links for Go and C#? For example: https://github.com/pulumi/examples/blame/master/README.md#L60

aws-cs-assume-role/README.md Show resolved Hide resolved
aws-cs-assume-role/assume-role/AssumeRoleStack.cs Outdated Show resolved Hide resolved
aws-cs-assume-role/create-role/CreateRoleStack.cs Outdated Show resolved Hide resolved
aws-cs-assume-role/create-role/CreateRoleStack.cs Outdated Show resolved Hide resolved
aws-cs-assume-role/assume-role/Pulumi.yaml Outdated Show resolved Hide resolved
aws-cs-assume-role/create-role/Program.cs Outdated Show resolved Hide resolved
aws-go-assume-role/assume-role/main.go Show resolved Hide resolved
aws-go-assume-role/create-role/main.go Outdated Show resolved Hide resolved
aws-ts-assume-role/create-role/index.ts Outdated Show resolved Hide resolved
aws-py-assume-role/create-role/venv/pyvenv.cfg Outdated Show resolved Hide resolved
@zephyrz73 zephyrz73 requested a review from justinvp July 14, 2020 19:01
Copy link
Member

@justinvp justinvp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like some of the feedback for aws-cs-assume-role/create-role/CreateRoleStack.cs wasn't addressed, but LGTM otherwise.

@zephyrz73 zephyrz73 merged commit c3d4581 into master Jul 15, 2020
@pulumi-bot pulumi-bot deleted the zephyrz73/assume-role-conversion-cs branch July 15, 2020 18:53
dixler pushed a commit that referenced this pull request Jan 21, 2022
Converted Assume Role to C# and also Fixed Secret Key Access to Non-Plain Text
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants