Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Plugs with long filenames will appear to upload but then get ignored #13

Open
hale6666 opened this issue Mar 11, 2022 · 1 comment
Open
Assignees

Comments

@hale6666
Copy link

When a plug has a long filename (in this case it was 89072973_2283623795273490_243277624934137856_n.jpg), it will appear to upload properly and show the confirmation page, but will then get ignored and not show up as pending approval.

@WillNilges WillNilges self-assigned this Mar 12, 2022
@WillNilges
Copy link
Member

WillNilges commented Mar 22, 2022

time="2022-03-22T04:32:22Z" level=info msg="current balance for %s is %dskyz264"
time="2022-03-22T04:32:27Z" level=error msg="pq: value too long for type character varying(64)"
time="2022-03-22T04:32:27Z" level=info msg="Uploaded new Plug!" plug_id=0 plug_s3id="2022/03/22/043222-skyz-Screen Shot 2022-03-21 at 11.58.25 PM_ccexpress.png" uid=skyz
[GIN] 2022/03/22 - 04:32:27 | 200 |  4.906004928s |    129.21.49.25 | POST     /upload
[GIN] 2022/03/22 - 04:32:42 | 200 |   14.505838ms |    129.21.49.25 | GET      /admin
time="2022-03-22T04:32:55Z" level=info msg="Changed Approved Plug List" plugs_approved="215,201,219,223,232,172,243,230,207,239,245,175,198,241,208,240,234,237,242,227,231,170,209,229,157,163,238,221,244,236,202,206,214,200,197,210,211,218,190" uid=wilnil

Found some logs. Looks like the SQL database can only handle file names of 64 characters or less. Should add a check before the user gets billed for the credits. Oops.

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

No branches or pull requests

2 participants