ci: update git push action for secure authentication #162

appleboy · 2023-04-28T02:39:00Z

appleboy commented

2023-04-28 02:39:00 +00:00

Replace the old git push action with a new one from a different repository
Remove commands for git configuration and committing/pushing code
Add new parameters for the git push action, including a commit message and remote repository information
Use a secret key for authentication in the git push action

Signed-off-by: appleboy appleboy.tw@gmail.com

- Replace the old git push action with a new one from a different repository - Remove commands for git configuration and committing/pushing code - Add new parameters for the git push action, including a commit message and remote repository information - Use a secret key for authentication in the git push action Signed-off-by: appleboy <appleboy.tw@gmail.com>

appleboy added 1 commit 2023-04-28 02:39:01 +00:00

ci: update git push action for secure authentication

- Replace the old git push action with a new one from a different repository
    - Remove commands for git configuration and committing/pushing code
    - Add new parameters for the git push action, including a commit message and remote repository information
    - Use a secret key for authentication in the git push action

Signed-off-by: appleboy <appleboy.tw@gmail.com>

test / test (3.6.14) (pull_request) Successful in 2m37s

Details

test / test (latest) (pull_request) Successful in 2m59s

Details

bdd1040cf0

appleboy requested review from wxiaoguang 2023-04-28 03:31:42 +00:00

wolfogre requested changes 2023-04-28 06:30:33 +00:00

.gitea/workflows/update.yaml Outdated

						
				@ -22,0 +20,4 @@

				          branch: main

				          commit: true

				          commit_message: "[skip ci] Updated Version via cron"

				          remote: "git@gitea.com:gitea/act_runner.git"

wolfogre commented

2023-04-28 06:30:29 +00:00

Why push to act_runner?

appleboy marked this conversation as resolved

appleboy added 1 commit 2023-04-28 12:27:44 +00:00

chore: improve API usage and test coverage across OSs

- Change remote repository in update.yaml from `act_runner.git` to `homebrew-gitea.git`

Signed-off-by: appleboy <appleboy.tw@gmail.com>

test / test (3.6.14) (pull_request) Successful in 2m32s

Details

test / test (latest) (pull_request) Successful in 2m54s

Details

71b00e5ee3

appleboy requested review from wolfogre 2023-04-28 12:28:16 +00:00

silverwind commented

2023-04-28 15:09:05 +00:00

I have my doubts about replacing a flexible and easily understandable series git commands with an additional abstraction layer that may have its own weird behaviours and which requires every contributor to learn it, vs. git which is much more common knowledge.

👍 1

wolfogre commented

2023-04-30 10:55:18 +00:00

BTW, the git commands will use the action context gitea.token for authentication, so they do not require an additional secret as deploy key.

BTW, the git commands will use the action context `gitea.token` for authentication, so they do not require an additional secret as deploy key.

techknowlogick merged commit 165a819ba6 into main

2023-09-05 15:00:58 +00:00

techknowlogick referenced this issue from a commit

2023-09-05 15:01:00 +00:00

ci: update git push action for secure authentication (#162)

Sign in to join this conversation.

No reviewers