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

Bad file name renaming #4811

Closed
mcastroSG opened this issue May 9, 2016 · 5 comments
Closed

Bad file name renaming #4811

mcastroSG opened this issue May 9, 2016 · 5 comments

Comments

@mcastroSG
Copy link

mcastroSG commented May 9, 2016

Expected behaviour

In case of rename an already synced file at a local folder to a bad file name an error is generated with a little briefing about it.

Actual behaviour

A general error is returned and overlay icon is not updated till several retries are done to a red X.

Steps to reproduce

  1. Rename a bad file name at Share sync folder (i.e 😄.jpg )
  2. Wait till sync starts

Server configuration

Server OS: Ubuntu
Server Version: 9.1.0 Alpha

Client configuration

Client OS: OSX El capitán 10.11.4
Client Version: Version 2.2.0-nightly20160504 (build 3331)

Client OS: Ubuntu 14.04 LTE
Client Version: Version 2.2.0nightly201654 (build 1587)

Client OS: Windows 81
Client Version: Version 2.2.0-nightly20160506 (build 6057)

@mcastroSG mcastroSG added this to the 2.3.0-next milestone May 9, 2016
@dragotin
Copy link
Contributor

dragotin commented May 9, 2016

What is the bad filename? It is not asdf I hope ;-)

@mcastroSG
Copy link
Author

mcastroSG commented May 9, 2016

:smile:.txt for example, I´ve been checking with some mojis included at filename to generate some errors 🙈 . asdf is the name before rename to ':smile:.txt'

@mcastroSG mcastroSG changed the title [Enhancement][MultiOS] Unify behaviour in case of bad file name included at Shared sync folder. [Enhancement][MultiOS] Bad file name renaming May 9, 2016
@jturcotte
Copy link
Member

The problem is that ideally we should rely on the server to give us errors about invalid filenames. But maybe we aren't encoding the filenames properly (or at all)?

/cc @ogoffart

@rperezb
Copy link

rperezb commented May 10, 2016

Ideally, errors in invalid filenames management rely on the server so that "special cases" such as some External storages are handled properly

@guruz
Copy link
Contributor

guruz commented Jul 31, 2017

Should be better with 2.3\4

@guruz guruz closed this as completed Jul 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants