Skip to content
This repository has been archived by the owner on Mar 13, 2021. It is now read-only.

Pull Request Target #104

Merged
merged 1 commit into from
Oct 14, 2020
Merged

Pull Request Target #104

merged 1 commit into from
Oct 14, 2020

Conversation

nebhale
Copy link
Member

@nebhale nebhale commented Oct 14, 2020

Previously, the actions tied to pull requests executed on the pull_request event, which resulted in them not having access to the secrets required for them to execute. At the beginning of August, GitHub added the pull_request_target event which operates in a similar way except only actions in the base commit would execute ensuring that it was safe to expose secrets. This change updates the pull request actions to use pull_request_target, ensuring that they are always functional.

Previously, the actions tied to pull requests executed on the pull_request
event, which resulted in them not having access to the secrets required for
them to execute.  At the beginning of August, GitHub added the
pull_request_target event which operates in a similar way except only actions
in the base commit would execute ensuring that it was safe to expose secrets.
This change updates the pull request actions to use pull_request_target,
ensuring that they are always functional.

Signed-off-by: Ben Hale <bhale@vmware.com>
@nebhale nebhale requested a review from a team as a code owner October 14, 2020 14:15
@nebhale nebhale added semver:patch A change requiring a patch version bump type:task A general task labels Oct 14, 2020
@nebhale nebhale merged commit ce62fe4 into main Oct 14, 2020
@nebhale nebhale deleted the pull-request-target branch October 14, 2020 14:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
semver:patch A change requiring a patch version bump type:task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant