Skip to content

docs: Rename "unsecure" to insecure - #270

Merged
woodruffw merged 2 commits into
zizmorcore:mainfrom
szepeviktor:insecure-fix
Dec 10, 2024
Merged

docs: Rename "unsecure" to insecure#270
woodruffw merged 2 commits into
zizmorcore:mainfrom
szepeviktor:insecure-fix

Conversation

@szepeviktor

Copy link
Copy Markdown
Contributor

Closes #269

Comment thread docs/audits.md Outdated

However, users can explicitly re-enable them by setting the
`ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable at the workflow,
`ACTIONS_ALLOW_INSECURE_COMMANDS` environment variable at the workflow,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We can't change the environment variable anywhere, only the other uses of "unsecure" -- the envvar is GitHub's own misspelling, which we need to detect.

(Sorry, I didn't notice this in your search earlier.)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay!

Done 🍏

We have 1×unsecure, I've added some "GitHub"-s.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@woodruffw woodruffw left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you @szepeviktor!

@woodruffw woodruffw self-assigned this Dec 10, 2024
@woodruffw woodruffw added the documentation Improvements or additions to documentation label Dec 10, 2024
@woodruffw woodruffw changed the title Rename "unsecure" to insecure docs: Rename "unsecure" to insecure Dec 10, 2024
@woodruffw
woodruffw merged commit 198fd0d into zizmorcore:main Dec 10, 2024
@szepeviktor

Copy link
Copy Markdown
Contributor Author

Glad to contribute.

@szepeviktor
szepeviktor deleted the insecure-fix branch December 10, 2024 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: "Unsecure" is really insecure

2 participants