Skip to content

bug: Attachments are not displayed on issues #11270

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

Closed
1 of 3 tasks
bkraul opened this issue May 1, 2020 · 8 comments
Closed
1 of 3 tasks

bug: Attachments are not displayed on issues #11270

bkraul opened this issue May 1, 2020 · 8 comments
Labels
Milestone

Comments

@bkraul
Copy link

bkraul commented May 1, 2020

Description

Attachments added to an issue as part of comments are not being displayed. If an attachment is added by itself, the comment on the issue reads "Attached file".

Screenshots

Attachments gif

@lunny
Copy link
Member

lunny commented May 2, 2020

Will be resolved by #11272

@lunny lunny added this to the 1.12.0 milestone May 2, 2020
@6543
Copy link
Member

6543 commented May 2, 2020

should be closed now - @bkraul can you test if it works now on latest master build?

@bkraul
Copy link
Author

bkraul commented May 2, 2020

Sure thing man, as soon as CI builds the docker image:

image

Thanks for the quick action! 👍

@bkraul
Copy link
Author

bkraul commented May 2, 2020

Ok, so try.gitea.com has been updated, and the attachments "work" to an extent. Picture attachments seem to be working OK, but uploading other formats does not work. For example, I am trying to upload a docx file. It does not show up or there is any trace of it being uploaded. I know it is supported:

image

image

@lunny
Copy link
Member

lunny commented May 2, 2020

Why did you think try.gitea.io support pdf and docx? I think it only supports images and zip files.

@bkraul
Copy link
Author

bkraul commented May 2, 2020

I sent you a screenshot of what try.gitea.io says it supports (granted it is kind of weird that docx is put together with zip Archive..). And I know it does support PDFs and docx as well. Don't quite remember if there is a setting for MIME types that has to be configured for handling. But those were working on previous versions (pdf and docx).

@bkraul
Copy link
Author

bkraul commented May 2, 2020

And yes, I can see there is a setting on app.ini for this on app.ini. My local one is set to:

[attachment]
PATH          = /data/gitea/attachments
ALLOWED_TYPES = */*

But I have not tried it there because the build was not ready, will try now.

@bkraul
Copy link
Author

bkraul commented May 2, 2020

OK, I can verify the attachment of different file types once again works on my environment (because of my allowed types setting). I think you can close this issue. However, on try.gitea.io you might want to look at that upload component, I am not sure it is in sync with what's actually allowed (because it says it allows docx). 😄

Thanks again!

@bkraul bkraul closed this as completed May 2, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants