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

Fixing gitspiegel trigger workflow #141

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

mutantcornholio
Copy link
Contributor

The first attept to use a workflow to protect GitLab CI from untrusted contributors failed, because GitHub doesn't pass secrets to workflows for PRs that originate from forks.

This uses a different approach: instead of triggerring gitspiegel API directly from the workflow, we're just spawning an empty workflow with a specific path, and gitspiegel listens for workflow_run event to start mirroring.

The idea is the same: for the first-time contributors, running workflows would require manual aciton and that would block mirroring. But this time, we don't need any secrets to make it work.

The first attept to use a workflow to protect GitLab CI from untrusted contributors failed, because GitHub doesn't pass secrets to workflows for PRs that originate from forks. 
 
This uses a different approach: instead of triggerring gitspiegel API directly from the workflow, we're just spawning an empty workflow with a specific path, and gitspiegel listens for `workflow_run` event to start mirroring.  

The idea is the same: for the first-time contributors, running workflows would require manual aciton and that would block mirroring. But this time, we don't need any secrets to make it work.
@mutantcornholio mutantcornholio requested a review from a team as a code owner November 7, 2023 16:55
@mutantcornholio mutantcornholio merged commit 184cea4 into master Nov 7, 2023
@mutantcornholio mutantcornholio deleted the yuri/gitspiegel-trigger-fix branch November 7, 2023 17:29
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