Skip to content

fetchgit: remove "security" from comment about "security risk"#185005

Merged
Artturin merged 1 commit intomasterfrom
unknown repository
Jan 14, 2023
Merged

fetchgit: remove "security" from comment about "security risk"#185005
Artturin merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 3, 2022

Description of changes

Closes #178410

Things done

@github-actions github-actions bot added the 6.topic: fetch Fetchers (e.g. fetchgit, fetchsvn, ...) label Aug 3, 2022
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Aug 3, 2022
@bobby285271 bobby285271 added the 12.approvals: 1 This PR was reviewed and approved by one person. label Aug 4, 2022
@mweinelt mweinelt added 12.approvals: 2 This PR was reviewed and approved by two persons. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Aug 5, 2022
@Artturin Artturin merged commit 1121c8f into NixOS:master Jan 14, 2023
@ghost ghost deleted the pr/fetchgit/fix-comment branch January 15, 2023 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: fetch Fetchers (e.g. fetchgit, fetchsvn, ...) 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 12.approvals: 2 This PR was reviewed and approved by two persons.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"fetching arbitrary (maybe dangling) commits may be a security risk"

5 participants