Skip to content

* Generate credentials updated to a post request. #430

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

Merged
merged 2 commits into from
Apr 24, 2019

Conversation

JadeHayes
Copy link
Contributor

@JadeHayes JadeHayes commented Apr 24, 2019

  • Myself and https://github.com/hvac/hvac/pull/403/files received an error when passing in TTL. GET requests do not recognize json, so the passed in TTL always defaulted to 1h.

  • This PR uses a POST request to call the endpoint.

@JadeHayes JadeHayes changed the title * Removed if statement for sts. * Generate credentials updated to a post request. Apr 24, 2019
@codecov-io
Copy link

codecov-io commented Apr 24, 2019

Codecov Report

Merging #430 into develop will increase coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #430      +/-   ##
===========================================
+ Coverage    90.11%   90.14%   +0.03%     
===========================================
  Files           46       46              
  Lines         2408     2406       -2     
===========================================
- Hits          2170     2169       -1     
+ Misses         238      237       -1
Impacted Files Coverage Δ
hvac/api/secrets_engines/aws.py 100% <100%> (+1.85%) ⬆️

@jeffwecan
Copy link
Member

Thanks @JadeHayes!

@jeffwecan jeffwecan merged commit cce5b86 into hvac:develop Apr 24, 2019
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.

3 participants