dikhan / terraform-provider-openapi

OpenAPI Terraform Provider that configures itself at runtime with the resources exposed by the service provider (defined in a swagger file)
Apache License 2.0
275 stars 48 forks source link

[TechDebt: Issue #253] Add gosec to CI/CD pipeline and address gosec alerts #254

Closed lillchan closed 4 years ago

lillchan commented 4 years ago

What problem does this Pull Request solve?

Please link to the issue number here (issue will be closed when PR is merged): Closes #253

If you don't have an issue created, please create the issue first describing the problem and then open the PR

Type of change

What type of change does your code introduce to the provider? Please put an x (w/o heading/trailing white spaces) in the boxes that apply:

Checklist

Please put an x (w/o heading/trailing white spaces) in the boxes that apply:

Checklist for Admins

codecov[bot] commented 4 years ago

Codecov Report

Merging #254 into master will decrease coverage by 0.01%. The diff coverage is 91.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #254      +/-   ##
==========================================
- Coverage   91.88%   91.87%   -0.02%     
==========================================
  Files          62       61       -1     
  Lines        3685     3593      -92     
==========================================
- Hits         3386     3301      -85     
+ Misses        201      198       -3     
+ Partials       98       94       -4     
Impacted Files Coverage Δ
openapi/openapi_stub_resource.go 100.00% <ø> (ø)
openapi/openapi_v2_security.go 95.45% <ø> (ø)
openapi/plugin_config.go 71.83% <0.00%> (ø)
openapi/provider.go 58.33% <ø> (ø)
openapi/utils.go 60.00% <ø> (ø)
openapi/common.go 90.05% <100.00%> (+0.11%) :arrow_up:
...n_config_services_schema_property_configuration.go 96.92% <100.00%> (+0.09%) :arrow_up:
openapi/resource_factory.go 92.44% <100.00%> (+0.29%) :arrow_up:
tests/e2e/test_api_utils.go 60.71% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0f896de...521bd03. Read the comment docs.