Skip to content

Fix IE11 upload fallback methods - #20595

Merged
skjnldsv merged 1 commit into
masterfrom
fix/ie11-fallback-upload
Apr 22, 2020
Merged

Fix IE11 upload fallback methods#20595
skjnldsv merged 1 commit into
masterfrom
fix/ie11-fallback-upload

Conversation

@skjnldsv

@skjnldsvskjnldsv commented Apr 22, 2020

Copy link
Copy Markdown
Member

😡

https://developer.mozilla.org/en-US/docs/Web/API/FileReader/readAsBinaryString#Browser_compatibility

Steps:

  • Try uploading a file without extension, see an error without any more informations

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable18

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable17

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable16

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙈

@nickvergessen

Copy link
Copy Markdown
Member

@ma12-co do we need this in talk too?

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

@ma12-co do we need this in talk too?

Now I'm afraid to look at the code you're using in talk 🙈

@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 22, 2020

@danxuliudanxuliu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

However, note that the problem was not in files without extension, but in files without extension, with size divisible by 4096 and smaller than 102400 bytes. How did you find this? xD

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

How did you find this? xD

I have my reasons xD

@skjnldsv
skjnldsv merged commit d0a57a3 into masterApr 22, 2020
@skjnldsv
skjnldsv deleted the fix/ie11-fallback-upload branch April 22, 2020 20:06
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable18 in #20602

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable17 in #20603

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable16 in #20604

@rullzerrullzer mentioned this pull request Apr 22, 2020
55 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@skjnldsv@nickvergessen@MorrisJobke@juliusknorr@danxuliu