1 Topic by davejsmith

  • davejsmith
  • Member
  • Offline
  • Registered: 2012-04-27
  • Posts: 1

Topic: Chrome Download from .Net mcFileManager Bug

Hi,
I'm using the .Net version of mcFileManager integrated into a web application, all works fine except for when downloading any file using Google Chrome which always seems to see the filename as default.aspx although the actual file content seems OK.
Looking at the http response header using Fiddler I can see than an extra colon (:) has crept into the content disposition header, other browsers don't seem to mind this but it causes this issue in Chrome.

Content-Disposition: : attachment; filename="649877_37690032.jpg"

Thanks

Dave.

2 Reply by Afraithe

  • Afraithe
  • Administrator
  • Offline
  • From: Skellefteå, Sweden
  • Registered: 2005-01-03
  • Posts: 2,657

Re: Chrome Download from .Net mcFileManager Bug

Hmm, strange, will look into that.

Afraithe
TinyMCE Developer
Moxiecode Systems

Afraithe's Website

3 Reply by munyung

  • munyung
  • Member
  • Offline
  • Registered: 2012-12-17
  • Posts: 1

Re: Chrome Download from .Net mcFileManager Bug

Hi, any findings on this issue? i am experiencing it as well and my client wants to know if there is a fix yet.

4 Reply by Afraithe

  • Afraithe
  • Administrator
  • Offline
  • From: Skellefteå, Sweden
  • Registered: 2005-01-03
  • Posts: 2,657

Re: Chrome Download from .Net mcFileManager Bug

Yes, its true an extra : snuck into the code.

You can find it in src/csharp/Manager/CorePlugin.cs on like 197

context.Response.AddHeader("Content-Disposition:", "attachment; filename=\"" + file.Name + "\"");

should be

context.Response.AddHeader("Content-Disposition", "attachment; filename=\"" + file.Name + "\"");

Ofc you need to recompile the DLL for this.

We aren't releasing an update right now just for this.

Afraithe
TinyMCE Developer
Moxiecode Systems

Afraithe's Website

5 Reply by KezberDev

  • KezberDev
  • Member
  • Offline
  • Registered: 2013-02-08
  • Posts: 1

Re: Chrome Download from .Net mcFileManager Bug

Hi, we have the same problem.
How can we get access to the source code mentionned above?

Thank you,

6 Reply by Afraithe

  • Afraithe
  • Administrator
  • Offline
  • From: Skellefteå, Sweden
  • Registered: 2005-01-03
  • Posts: 2,657

Re: Chrome Download from .Net mcFileManager Bug

Source is in the package, you should just be able to make the above change and compile it.

Afraithe
TinyMCE Developer
Moxiecode Systems

Afraithe's Website

7 Reply by GiancarloGomez

  • GiancarloGomez
  • Member
  • Offline
  • Registered: 2013-04-06
  • Posts: 1

Re: Chrome Download from .Net mcFileManager Bug

Can you guys please post a fix as I am having trouble compiling this project since I am using VS 2012 on a W8 machine.

8 Reply by Afraithe

  • Afraithe
  • Administrator
  • Offline
  • From: Skellefteå, Sweden
  • Registered: 2005-01-03
  • Posts: 2,657

Re: Chrome Download from .Net mcFileManager Bug

GiancarloGomez, we are going to push out a maintnance release of the .NET version, planned next week.

Afraithe
TinyMCE Developer
Moxiecode Systems

Afraithe's Website