google / uuid

Go package for UUIDs based on RFC 4122 and DCE 1.1: Authentication and Security Services.
BSD 3-Clause "New" or "Revised" License
5.29k stars 363 forks source link

Limit permissions for github workflows #142

Closed joycebrum closed 8 months ago

joycebrum commented 10 months ago

Hi! I'm from GOSST (Google Open Source Security Team) team and I'd like to contribute to uuid to in order to improve its supply-chain security.

My suggestion here is basically set up the permissions to the github workflows as read only on the top level and any write permission be given at the run level.

This is necessary due to a behavior of github workflow to grant to GITHUB_TOKEN write permissions to all types of permissions, regardless of they being used or not. In case of the workflow getting compromised, an attacker can exploit this permissions.

Thus, it is both a recommendation from OpenSSF Scorecard and the Github to always use credentials that are minimally scoped.

I'll soon submit a PR to show the changes I listed above. Please, feel free to reach me out in case of any doubts or concerns.