Opened 9 years ago
Closed 8 years ago
#10829 closed Patch (fixed)
[new feature] Option to copy URL with password
Reported by: | Julien Blitte | Owned by: | |
---|---|---|---|
Priority: | normal | Component: | FileZilla Client |
Keywords: | client URL CServer FormatServer | Cc: | |
Component version: | 7473 | Operating system type: | |
Operating system version: |
Description
Hi,
This patch is a modification proposal to add the option to include the password when copying an URL (mainly for FTP protocol).
The option has to be enabled by user, and before activation a security warning is of course shown.
I know this option can be seen as unsafe but, I really need it: I am dealing with 4-5 different FTPs all the day and I have to add password back manually each time I want to send a FTP link to one of my customer.
Modification are made on top of revision 7470.
Main modification is the method FormatServer of CServer.
Warning: I am sorry but I did this patch without compiling and testing it, so next steps would be try to compile with it and to debug it.
If feature is approved, it should not be hard to debug it anyway, I am not bad in C/C++ but there will be obviously some bugs/typo as just added some piece of code with notepad in few files.
Julien
Attachments (3)
Change History (9)
by , 9 years ago
Attachment: | Option to copy URL with password.patch added |
---|
follow-up: 2 comment:1 by , 9 years ago
Status: | new → moreinfo |
---|
Please fix the patch such that the result compiles.
by , 9 years ago
Attachment: | Option to copy URL with password_2.patch added |
---|
Patch - Option to copy URL with password (compiles)
comment:2 by , 9 years ago
Status: | moreinfo → new |
---|
Replying to codesquid:
Please fix the patch such that the result compiles.
Done. I make a clean version that compiles and almost* works.
Patch is upon version 7473 now.
*Sorry, I have one an WX issue remaining at execution (located in src/interface/Mainfrm.cpp:840 - when I try to untick the menu), I did not manage to fix it yet.
The rest seems to work. I upload another version without the error.
by , 9 years ago
Attachment: | Option to copy URL with password_3.patch added |
---|
Patch - Option to copy URL with password (compiles, full works and run without error)
comment:3 by , 9 years ago
Component version: | 7470 → 7473 |
---|
comment:5 by , 8 years ago
I've committed some of the changes from your patch with a few modifications:
- Dropped the superfluous extra #includes
- Renamed the new enum and made it a strongly-typed one
So far I haven't decided what to do about the GUI portion. An extra item in the main menu just to toggle passwords in a contextual feature feels like really bad UI design.
I need to do something about the URL encoding as well.
comment:6 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Patch integrated with the great enhancements from Tim.
Thanks !
New functionality: Starting 3.18.0 RC, pressing [Shift] while doing a right click will enable option to copy with password.
Internal code changes : CServer / FormatServer and URL encoding also done by Tim as far as I saw in the code.
Julien Blitte
Patch - Option to copy URL with password