-
-
Notifications
You must be signed in to change notification settings - Fork 135
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
AWS.GenerateSTSCredentialsAsync() should use GET instead of POST (according to vault docs) #148
Conversation
Turns out VaultSharp is using a non-standard access moethod for generating AWS STS access tokens. The documentation says GET https://www.vaultproject.io/api-docs/secret/aws#generate-credentials but VaultSharptoday uses a POST call
@rajanadar Would it be possible to get this merged and a release created? I've created an internal forked release in the meantime but would be nice to get back to using the main project. |
@megakid no so far ))) |
thanks @icanhasjonas for the PR. @megakid I have published a new version with this fix. Please test and let me know if any issues. |
Let me explain one last time, since human beings exist under different context. I respect all the time and effort in your contribution for this project. And your feedback and dedication. So I am hoping that, this is the last time we have wisecrack statements or taunts :). From one student of software to another. Have a great day! Cheers. |
@rajanadar If you can't do that, let others do it |
@Marusyk that is very harsh and uncalled for. This is an open source library, not something that you should expect support for. @rajanadar sorry to hear about your emergency- I hope things are going ok now. I appreciate you spending the time to get a new release out, I will try to test it next week. |
Look @megakid, you don't know the full history since 2019! I'm not talking about direct support. What do you think @rajanadar? |
thanks @megakid. Please let me know if you see any issue. |
Turns out
VaultSharp
is using a non-standard access method for generating AWS STS access tokens. The documentation saysGET
https://www.vaultproject.io/api-docs/secret/aws#generate-credentials butVaultSharp
today uses aPOST
call