repo-file-sync-action icon indicating copy to clipboard operation
repo-file-sync-action copied to clipboard

Published 20 hours ago • verified publisher icon BetaHuhn

Problem setting git username/email with GH_PAT only

Open ndom91 opened this issue 2 years ago • 9 comments

🐞 Describe the bug

Our sync action stopped working ~5 days ago (seemingly after this was merged - https://github.com/BetaHuhn/repo-file-sync-action/pull/153). Throwing an error that the git username/email was missing.

We're passing only a GH_PAT, not an installation token.

🖼️ Screenshots

image

See last working run: https://github.com/nextauthjs/next-auth/actions/runs/1809749953 Failing run: https://github.com/nextauthjs/next-auth/actions/runs/1815281481

📋 Additional context

Sync action: https://github.com/nextauthjs/next-auth/blob/main/.github/workflows/sync-example.yml Sync config: https://github.com/nextauthjs/next-auth/blob/main/.github/sync.yml

ndom91 avatar Feb 13 '22 18:02 ndom91

Hi @ndom91, thanks for reporting this!

I will take a look and get this fixed asap.

BetaHuhn avatar Feb 14 '22 19:02 BetaHuhn

I can't seem to reproduce your issue. I tried running the action with only the GH_PAT and it is working without any errors (see the test run).

Could you enable the step debug logging by setting the ACTIONS_STEP_DEBUG secret to true and try again? It should print that it is setting the git username and email.

BetaHuhn avatar Feb 14 '22 20:02 BetaHuhn

I can't seem to reproduce your issue. I tried running the action with only the GH_PAT and it is working without any errors (see the test run).

Could you enable the step debug logging by setting the ACTIONS_STEP_DEBUG secret to true and try again? It should print that it is setting the git username and email.

Thanks for taking a look! We'll give that a try and get back to you.

Cc: @balazsorban44

ndom91 avatar Feb 14 '22 23:02 ndom91

Yes, we can try that.

I got 404 on your test run, by the way. Is it a hidden repository you link to?

FWIW, pinning down to the previous version worked:

https://github.com/nextauthjs/next-auth/actions/runs/1838645699

https://github.com/nextauthjs/next-auth/commit/1c2b57393396b9a7611cea38d2ca087d411c5bd6

balazsorban44 avatar Feb 15 '22 12:02 balazsorban44

I got 404 on your test run, by the way. Is it a hidden repository you link to?

Whoops, yeah it was set to private. Fixed now.

BetaHuhn avatar Feb 15 '22 13:02 BetaHuhn

Just checking in, is the issue still coming up? @balazsorban44 @ndom91

BetaHuhn avatar Mar 13 '22 09:03 BetaHuhn

Closing this for now. Please reopen it if the issue persists.

BetaHuhn avatar May 05 '22 08:05 BetaHuhn

I'm experiencing this issue on latest

aehlke avatar Jun 14 '22 13:06 aehlke

Also experiencing this issue on latest, using GH_PAT only with the same error message.

SignorMercurio avatar Jul 19 '22 04:07 SignorMercurio

I'm experiencing this issue on latest as of 10/23/2022

JBG-brad avatar Oct 23 '22 17:10 JBG-brad