rollbar / terraform-provider-rollbar

Terraform provider for Rollbar
https://rollbar.com
MIT License
28 stars 13 forks source link

sanitize id for project access token resource #409

Closed pawelsz-rb closed 2 months ago

pawelsz-rb commented 2 months ago

Description of the change

terraform ID for a project access token resource was set to access token, which was not really secure causing a leaking token. Now it's sanitized by computing md5 sum on that token. Change was tested, it's backward compatible.

Type of change

Checklists

Development

Code review

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 10678663426

Details


Totals Coverage Status
Change from base Build 9972229186: 0.0%
Covered Lines: 1480
Relevant Lines: 1607

💛 - Coveralls
codeclimate[bot] commented 2 months ago

Code Climate has analyzed commit 32838f24 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 92.0% (0.0% change).

View more on Code Climate.