-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support GITHUB_TOKEN provided by GitHub Actions #4176
Comments
I encoded it into hex to be able to extract the integration token from the output (they block base64-based exfiltration though) and the format is The regex needs to be updated to allow the period.
|
For a temporary, cross-platform fix, a step like this should allow the new token format to work. Since sed exits with code - name: Temporarily fix electron-builder
run: |
sed -i 's/\^\[\\w/\^\[.\\w/g' node_modules/builder-util/out/util.js
shell: bash |
thanks |
On MacOS you need the following syntax for sed to work - name: Temporarily fix electron-builder
run: |
sed -i '' 's/\^\[\\w/\^\[.\\w/g' node_modules/builder-util/out/util.js
shell: bash |
@develar can we get a release with that fix? |
This reverts commit 52f3406. Until electron-userland/electron-builder#4176 is fixed
This is still happening with v21.2.0 (GitHub Actions on Ubuntu):
|
Workaround required because `electron-builder` considers GitHub Actions tokens invalid: electron-userland/electron-builder#4176
@samuelmeuli in https://github.com/samuelmeuli/mini-diary/blob/master/.github/workflows/release.yml#L48 you use a |
@loopmode Yes, that's what I did for now |
Ah, I see, the version that fixes this issue is not marked as stable yet, which is why it's not in my project. Running |
The same issue here:
|
I'm attempting to build an Electron app using GitHub Actions. A GITHUB_TOKEN is provided which could possibly be used instead of creating a personal access token.
However, the following error occurs:
The text was updated successfully, but these errors were encountered: