wefork / wekan

The open-source Trello-like kanban (built with Meteor)
https://wekan.io
MIT License
61 stars 12 forks source link

Can't download attachment with special character in file name #97

Closed nuali closed 6 years ago

nuali commented 7 years ago

Hey,

when I try to download an attachment from a card which has a special character in the file name (in my case an Ü), I get the following error:

TypeError: The header content contains invalid characters
    at ServerResponse.OutgoingMessage.setHeader (_http_outgoing.js:363:13)
    at ServerResponse.res.setHeader (/programs/server/npm/node_modules/meteor/webapp/node_modules/connect/lib/patch.js:134:22)
    at Proxy.translateResponse (server/proxy.js:1779:18)
    at server/proxy.js:2016:21
    at /programs/server/npm/node_modules/meteor/promise/node_modules/meteor-promise/fiber_pool.js:32:39

The reason seems to be an node.js update: https://www.alexkras.com/typeerror-the-header-content-contains-invalid-characters/

Can you please help me with that problem? :)

Thanks and sunny greetings!

nuali commented 7 years ago

Sorry, I checked the closed issues but somehow I missed this one: https://github.com/wefork/wekan/issues/87

xet7 commented 7 years ago

@nuali

Do you have newest version of Wekan? Are you using MongoDB 3.2.x ?

I think this bug is already fixed.

nuali commented 7 years ago

@xet7 I'm using it with sandstorm.

xet7 commented 7 years ago

@nuali

Wefork has been merged back to Wekan.

Please wait that new Wekan version release is made for Sandstorm, it's in progress, and it will include fixes, new features.etc.

nuali commented 7 years ago

@xet7 Alright, thanks, I just used the issue button inside the app, this is how I came here. :)

ocdtrekkie commented 7 years ago

@nuali That link will get fixed next time a release is made for the Sandstorm version as well. :)

nuali commented 6 years ago

Actually, it wasn't fixed in the newest release 0.47.0~2017-10-04 :(

xet7 commented 6 years ago

Fixed already.