Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

update secops orb #59

Merged
merged 1 commit into from
Oct 19, 2023
Merged

update secops orb #59

merged 1 commit into from
Oct 19, 2023

Conversation

peakematt
Copy link
Contributor

Motivation / Implements

This PR updates the version of the SecOps orb used in this repo. Version 2.0.0 of the orb included an issue on the gitleaks job that prevented the job from running on PRs created from forks. The issue was caused by a default configuration in CircleCI which prevents providing secrets provided through CircleCI contexts to PRs created from forks. This default configuration is probably the correct one, so we needed up update the SecOps orb to properly operate on PRs from forks.

@peakematt peakematt marked this pull request as ready for review October 19, 2023 19:10
@peakematt peakematt requested a review from dariuszkuc as a code owner October 19, 2023 19:10
@peakematt
Copy link
Contributor Author

@dariuszkuc Apologies for the quick follow-up PR on this.

@dariuszkuc dariuszkuc merged commit d9fffb4 into main Oct 19, 2023
@dariuszkuc dariuszkuc deleted the peake/update-secops-orb branch October 19, 2023 19:12
SimonSapin pushed a commit to apollographql/router that referenced this pull request May 3, 2024
## Motivation / Implements
This PR updates the version of the SecOps orb used in this repo. Version
2.0.0 of the orb included an issue on the gitleaks job that prevented
the job from running on PRs created from forks. The issue was caused by
a default configuration in CircleCI which prevents providing secrets
provided through CircleCI contexts to PRs created from forks. This
default configuration is probably the correct one, so we needed up
update the SecOps orb to properly operate on PRs from forks.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants