Skip to content

Standards sec fix#2

Merged
klamb merged 3 commits into
mainfrom
standards-sec-fix
Apr 14, 2026
Merged

Standards sec fix#2
klamb merged 3 commits into
mainfrom
standards-sec-fix

Conversation

@qiuz-cz
Copy link
Copy Markdown
Contributor

@qiuz-cz qiuz-cz commented Apr 13, 2026

No description provided.

qiuz-cz added 3 commits April 13, 2026 17:35
…PDX headers

- Replace BSD 3-Clause with Apache License 2.0 in LICENSE
- Add NOTICE with CloudZero attribution
- Add CONTRIBUTING.md (inbound licensing, dev setup)
- Add SECURITY.md (private vulnerability reporting)
- Update README: Apache 2.0 license section + trademark notice
- Add SPDX-FileCopyrightText / SPDX-License-Identifier headers to all 21 Go source files
- Add .env to .gitignore
…st_key

- Add permissions: contents: read to test workflow (least privilege)
- Pin tfplugindocs to v0.24.0 (was @latest, supply chain risk)
- Mark test_key as Sensitive: true in provider schema
Mutable semver tags can be force-pushed by a compromised upstream
maintainer. Pin all actions to full commit SHAs to prevent silent
tag-substitution supply chain attacks. Version tags preserved in
comments for readability.

- actions/checkout@v4       => 34e1148
- actions/setup-go@v5       => 40f1582
- golangci-lint-action@v7   => 9fae48a
- ghaction-import-gpg@v6    => e89d409  (release.yml — has GPG secrets)
- goreleaser-action@v6      => e435ccd  (release.yml — has write permissions)
Copy link
Copy Markdown
Collaborator

@klamb klamb left a comment

Choose a reason for hiding this comment

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

Looks good. License upgrade to Apache 2.0 aligns with Terraform provider ecosystem conventions. Security hardening (pinned actions, least-privilege permissions, sensitive fields, SPDX headers) all appropriate for a public-facing provider.

@klamb klamb merged commit 49504f9 into main Apr 14, 2026
4 checks passed
@qiuz-cz qiuz-cz deleted the standards-sec-fix branch May 1, 2026 01:02
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.

2 participants