Skip to content
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

Error: Request body length does not match content-length header #556

Open
qwerttvv opened this issue Dec 11, 2024 · 16 comments
Open

Error: Request body length does not match content-length header #556

qwerttvv opened this issue Dec 11, 2024 · 16 comments

Comments

@qwerttvv
Copy link

    - name: Upload firmware to release
      uses: softprops/action-gh-release@master
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      with:
        tag_name: ${{ steps.tag.outputs.release_tag }}
        body_path: release.txt
        files: ${{ env.FIRMWARE }}/*

full yml
https://github.com/qwerttvv/Router/blob/master/.github/workflows/M2-VIKINGYFY.yml

log

👩‍🏭 Creating new GitHub release for tag 2024.12.11-22.51_M2-VIKINGYFY...
⬆️ Uploading version.buildinfo...
⬆️ Uploading sha256sums...
⬆️ Uploading profiles.json...
⬆️ Uploading immortalwrt-qualcommax-ipq60xx-cmiot_ax18.manifest...
⬆️ Uploading immortalwrt-qualcommax-ipq60xx-cmiot_ax18-squashfs-sysupgrade.bin...
⬆️ Uploading immortalwrt-qualcommax-ipq60xx-cmiot_ax18-squashfs-factory.ubi...
⬆️ Uploading immortalwrt-qualcommax-ipq60xx-cmiot_ax18-initramfs-uImage.itb...
⬆️ Uploading feeds.buildinfo...
⬆️ Uploading config.buildinfo...
Error: Request body length does not match content-length header

good ver-last week
https://github.com/softprops/action-gh-release/tree/92bc83c4214f780e84c7d2a85464a2ca05ebc2f0

not good ver-nowsoftprops/action-gh-release@master
https://github.com/softprops/action-gh-release/tree/7b4da11513bf3f43f9999e90eabced41ab8bb048

@gtxaspec
Copy link

confirmed, the latest commits produce this error.

@wanansany
Copy link

Don't use master,use v2
uses: softprops/action-gh-release@v2

@qwerttvv
Copy link
Author

#555 smae issue

@VietND96
Copy link

Don't use master,use v2 uses: softprops/action-gh-release@v2

The same goes for me.
I saw the latest commit on master is releasing v2.2.0, so I am wondering why softprops/action-gh-release@v2 is able to work?

@qwerttvv
Copy link
Author

Don't use master,use v2 uses: softprops/action-gh-release@v2

The same goes for me. I saw the latest commit on master is releasing v2.2.0, so I am wondering why softprops/action-gh-release@v2 is able to work?

now, v2=v2.0.9
see this https://github.com/softprops/action-gh-release/tags

@VietND96
Copy link

Thanks for pointing. I assumed it always points to the latest, but need the author to bump it.

github-merge-queue bot pushed a commit to dfinity/ic that referenced this issue Dec 12, 2024
This change applies @nmattia's suggestion about using a default shell.
He also mentioned he believes that bash is the default but prior to this
I had to set the shell to bash so I'm not confident if that is true.

It also downgrades the release action due to
softprops/action-gh-release#556
papjul added a commit to breezy-weather/breezy-weather that referenced this issue Dec 12, 2024
bandoche added a commit to bandoche/GeoLite.mmdb that referenced this issue Dec 13, 2024
YummyBacon5 added a commit to likes-gay/Cave-of-Shadows that referenced this issue Dec 13, 2024
@brunoalr
Copy link

brunoalr pushed a commit to OpenImageDebugger/OpenImageDebugger that referenced this issue Dec 13, 2024
brunoalr pushed a commit to OpenImageDebugger/OpenImageDebugger that referenced this issue Dec 13, 2024
Nevuly added a commit to Nevuly/WSL2-Linux-Kernel-Rolling that referenced this issue Dec 15, 2024
Error Logs:
Run softprops/action-gh-release@master
Found release linux-wsl-stable-6.12.5 (with id=190802427)
⬆️ Uploading bzImage-x86_64-optional_install.tar.gz...
⬆️ Uploading bzImage-x86_64-modules_install.tar.gz...
⬆️ Uploading bzImage-x86_64...
⬆️ Uploading Image-arm64-optional_install.tar.gz...
⬆️ Uploading Image-arm64-modules_install.tar.gz...
⬆️ Uploading Image-arm64...
Error: Request body length does not match content-length header

 * `master` version of action-gh-release have problem to release packages. (softprops/action-gh-release#556)
 * So, I try to use `v2` version for avoid this problem.

Signed-off-by: Yang Jeong Hun <onyxclover9931@gmail.com>
Nevuly added a commit to Nevuly/WSL2-Linux-Kernel-Rolling-LTS that referenced this issue Dec 15, 2024
Error Logs:
Run softprops/action-gh-release@master
Found release linux-wsl-stable-6.12.5 (with id=190802427)
⬆️ Uploading bzImage-x86_64-optional_install.tar.gz...
⬆️ Uploading bzImage-x86_64-modules_install.tar.gz...
⬆️ Uploading bzImage-x86_64...
⬆️ Uploading Image-arm64-optional_install.tar.gz...
⬆️ Uploading Image-arm64-modules_install.tar.gz...
⬆️ Uploading Image-arm64...
Error: Request body length does not match content-length header

 * `master` version of action-gh-release have problem to release packages. (softprops/action-gh-release#556)
 * So, I try to use `v2` version for avoid this problem.

Signed-off-by: Yang Jeong Hun <onyxclover9931@gmail.com>
JanDeDobbeleer pushed a commit to JanDeDobbeleer/oh-my-posh that referenced this issue Dec 15, 2024
JuliusBairaktaris added a commit to JuliusBairaktaris/Qualcommax_NSS_Builder that referenced this issue Dec 16, 2024
Locietta added a commit to Locietta/xanmod-kernel-WSL2 that referenced this issue Dec 16, 2024
@clintonsteiner
Copy link

64f1fa1
Believe related to this one

@gtxaspec
Copy link

FYI @xen0n @chenrui333

@clintonsteiner
Copy link

https://github.com/jenkinsci/jenkins/actions/runs/12416637271

Heres a run illustrating the behavior,

@clintonsteiner
Copy link

Error: Request body length does not match content-length header

@xen0n
Copy link
Contributor

xen0n commented Dec 20, 2024

Hmm, I will look into this.

@clintonsteiner
Copy link

@xen0n since so many others have confirmed the same issue and are rolling back.

Can we instead revert back to commits at 2.1.0, and rerelease it?

Thinking this package will end up forgotten about and never updated should too many repos commit locks for python 2.10

@xen0n
Copy link
Contributor

xen0n commented Dec 20, 2024

@xen0n since so many others have confirmed the same issue and are rolling back.

Can we instead revert back to commits at 2.1.0, and rerelease it?

Thinking this package will end up forgotten about and never updated should too many repos commit locks for python 2.10

I don't have the permissions to make a release, so this would have to be done by @chenrui333 .

Temporary workaround:

- uses: softprops/action-gh-release@v2.1.0

clintonsteiner added a commit to clintonsteiner/jenkins that referenced this issue Dec 20, 2024
N-Storm added a commit to N-Storm/Openwrt_Builder that referenced this issue Dec 20, 2024
Change create release action version from master to v2.2.0 to fix softprops/action-gh-release#556
timja added a commit to jenkinsci/jenkins that referenced this issue Dec 21, 2024
This reverts commit c39d7d6.
Documented here-softprops/action-gh-release#556

Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
broglep added a commit to broglep/homeassistant-meshtastic that referenced this issue Dec 22, 2024
xen0n added a commit to xen0n/action-gh-release that referenced this issue Dec 23, 2024
This allows the uploads to finish without mismatched Content-Length,
likely because the original method implied a wrong body encoding or
something similar. Unfortunately a GitHub server API mock was not
readily available so I had to test manually with a barebones repository.

Fixes: softprops#555
Fixes: softprops#556
Signed-off-by: WANG Xuerui <git@xen0n.name>
@xen0n
Copy link
Contributor

xen0n commented Dec 23, 2024

Heads-up: I've made a fix at #562, waiting for merge and release. Sorry for the breakage!

ramonpetgrave64 added a commit to slsa-framework/slsa-github-generator that referenced this issue Dec 23, 2024
# Summary

fixes
#4031 (comment).

revert softprops/action-gh-release to v2.1.0

Folks suggest that 2.1.0 is the last known good version.
softprops/action-gh-release#556

## Testing Process

N/A

## Checklist

- [x] Review the contributing
[guidelines](https://github.com/slsa-framework/slsa-github-generator/blob/main/CONTRIBUTING.md)
- [x] Add a reference to related issues in the PR description.
- [x] Update documentation if applicable.
- [ ] Add unit tests if applicable.
- [ ] Add changes to the
[CHANGELOG](https://github.com/slsa-framework/slsa-github-generator/blob/main/CHANGELOG.md)
if applicable.
zerbina added a commit to zerbina/nimskull that referenced this issue Dec 24, 2024
….2.0"

The most-recent version of the action has a [bug](softprops/action-gh-release#556)
that results in big file uploads (and thus the "Publisher" workflow) to
fail

Reverts https://github.com/nim-works/nimskull#1479.
github-merge-queue bot pushed a commit to nim-works/nimskull that referenced this issue Dec 24, 2024
….2.0" (#1480)

## Summary

The most-recent version of the action has a
[bug](softprops/action-gh-release#556) that
results in big
file uploads (and thus the `publisher` workflow) failing.

Reverts #1479.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants