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

Request body larger than maxBodyLength limit #60

Closed
Milerius opened this issue Dec 5, 2020 · 5 comments · Fixed by #64
Closed

Request body larger than maxBodyLength limit #60

Milerius opened this issue Dec 5, 2020 · 5 comments · Fixed by #64

Comments

@Milerius
Copy link

Milerius commented Dec 5, 2020

Behaviour

Steps to reproduce this issue

https://github.com/smk762/atomicDEX-Desktop/runs/1502695126?check_suite_focus=true

🔔 Release event detected for refs/tags/0.3.1-beta2 in this workflow. Preparing to scan assets...
📦 1 asset(s) will be sent to VirusTotal for analysis.
⬇️ Downloading atomicdex-desktop-0.3.1-beta-windows.zip...
Error: Request body larger than maxBodyLength limit
@crazy-max
Copy link
Owner

crazy-max commented Dec 12, 2020

@Milerius I've pushed a fix to enhance error message output. Can you try again?
Also fyi, "The total payload size can not exceed 32MB." as per VirusTotal API specification.

umbynos added a commit to arduino/arduino-create-agent that referenced this issue Dec 22, 2020
…DED]: Request body larger than maxBodyLength limit`, should be fixed in v2.3.0 (for ref crazy-max/ghaction-virustotal#60)
@umbynos
Copy link

umbynos commented Dec 22, 2020

I'm having the same problem.
I'm trying to upload a binary to virustotal using your action.
This is the workflow I'm using:

name: released

on:
  release:
    types: [published]

jobs:
  virustotal:
    runs-on: ubuntu-latest
    steps:
      - name: VirusTotal Scan
        uses: crazy-max/ghaction-virustotal@v2.3.0
        with:
          vt_api_key: ${{ secrets.VIRUSTOTAL_API_KEY }}
          github_token: ${{ secrets.GITHUB_TOKEN }}
          update_release_body: true
          files: |
            .exe$

And the error is: Error: Cannot send ArduinoCreateAgent-1.2-windows-386-installer-chrome.exe to VirusTotal: Error [ERR_FR_MAX_BODY_LENGTH_EXCEEDED]: Request body larger than maxBodyLength limit

Before finding that issue I tried using V2 (as stated in the README.md) but the issue stays the same.
I'm trying to upload a 18.3 MB file, so it's well below the 32MB limit. I do not understand what's the problem

@crazy-max
Copy link
Owner

@umbynos Thanks for your feedback I will take a look

@crazy-max
Copy link
Owner

@umbynos Can you try with: crazy-max/ghaction-virustotal@fix-maxbodylength? Thanks.

@umbynos
Copy link

umbynos commented Dec 22, 2020

Seems to work just fine! Thanks a lot!! 🥂

crazy-max added a commit that referenced this issue Dec 22, 2020
Co-authored-by: CrazyMax <crazy-max@users.noreply.github.com>
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

Successfully merging a pull request may close this issue.

3 participants