WIP: Add token to config when creating new runner #97

Closed
Zettat123 wants to merge 1 commits from Zettat123/act_runner:local-private-reusable-workflow into main
Member

Partially fixes #91

Add token to runner config so the runner can access the reusable workflows from the same repository.

This PR is from #91 (comment) . Thanks @gjabell for the idea.

Partially fixes #91 Add token to runner config so the runner can access the reusable workflows from the same repository. This PR is from https://gitea.com/gitea/act_runner/issues/91#issuecomment-734544 . Thanks @gjabell for the idea.
Zettat123 added 1 commit 2023-04-04 09:14:39 +00:00
add token to config
All checks were successful
checks / check and test (pull_request) Successful in 1m22s
cd847e3e93
Author
Member

This can only allow a workflow to reuse the workflows from the same repository. As for the workflows from external private repositories, we need to do more work and make another PR to fix.

This can only allow a workflow to reuse the workflows from the same repository. As for the workflows from external private repositories, we need to do more work and make another PR to fix.
lunny approved these changes 2023-04-04 09:20:28 +00:00
wolfogre requested changes 2023-04-04 10:34:42 +00:00
wolfogre left a comment
Owner

Does it related to gitea/act#6 ?

The case is:

  • The actions repo is public on gitea.com
  • Clone it with a token from my_gitea.com
  • It will fail because the token is wrong even if the repo is public.
  • And gitea.com get a token not belonging to it, it can do bad thing now.
Does it related to https://gitea.com/gitea/act/pulls/6 ? The case is: - The actions repo is public on gitea.com - Clone it with a token from my_gitea.com - It will fail because the token is wrong even if the repo is public. - And gitea.com get a token not belonging to it, it can do bad thing now.
Author
Member

Does it related to gitea/act#6 ?

That's right. I'll rewrite this PR.

> Does it related to https://gitea.com/gitea/act/pulls/6 ? That's right. I'll rewrite this PR.
Zettat123 changed title from Add token to config when creating new runner to WIP: Add token to config when creating new runner 2023-04-04 10:43:01 +00:00
Author
Member

Made some improvements for getting reusable workflows from the same repository in gitea/act#38 . Looks like we don't need to change the logic in act_runner so I will close this PR.

Made some improvements for getting reusable workflows from the same repository in https://gitea.com/gitea/act/pulls/38 . Looks like we don't need to change the logic in `act_runner` so I will close this PR.
Zettat123 closed this pull request 2023-04-06 02:35:34 +00:00
All checks were successful
checks / check and test (pull_request) Successful in 1m22s
Required
Details

Pull request closed

Sign in to join this conversation.
No description provided.