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

2720 - Definitive Global/Country file fix, support for UTF-8 filenames #2752

Merged
merged 7 commits into from Jun 8, 2023

Conversation

rfontanarosa
Copy link
Member

@rfontanarosa rfontanarosa commented Jun 7, 2023

close #2720

upload: multer, due to busboy dependency, doesn't support utf-8 filenames by default. there is a workaround to fix it: expressjs/multer#1104

download: nodejs doesn't support utf-8 on response header content-disposition: https://stackoverflow.com/questions/23895924/expressjs-download-filename-utf-8

primary key: pgadmin doesn't allow me to change file table on the fly because it hadn't a primary key. this problem can occurs with others sql clients. I just added a primary key as in the other tables (col, row)

Registrazione.schermo.2023-06-07.alle.16.51.59.mov

@rfontanarosa rfontanarosa linked an issue Jun 7, 2023 that may be closed by this pull request
@rfontanarosa rfontanarosa self-assigned this Jun 7, 2023
@rfontanarosa rfontanarosa changed the title 2720 garbled file name in links repository 2720 - Definitive Global/Country file fix, support for UTF-8 filenames Jun 7, 2023
sorja
sorja previously approved these changes Jun 7, 2023
@sorja
Copy link
Contributor

sorja commented Jun 7, 2023

wrong video?

@Doc999tor
Copy link

There is a PR for Multer that solves the issue - expressjs/multer#1210

@mergify mergify bot merged commit a9afa11 into development Jun 8, 2023
5 checks passed
@mergify mergify bot deleted the 2720-garbled-file-name-in-links-repository branch June 8, 2023 11:18
@pengchenglai
Copy link
Contributor

@minotogna
@sorja
@rfontanarosa

Great to know it is fixed, plz move this file(Oportunidades de restauracion en Guatemala.pdf) to the Repository of Guatemala
Capture

Remove other two testing files:
2

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 this pull request may close these issues.

Garbled file name in Links & Repository
5 participants