Delicious Posts - social bookmarking fs-plugin

[HomoLogicus]

Delicious Posts is a mini bookmark manager that allows you to use and organize del.icio.us bookmarks directly from TC panels.

You can find screenshots and more information on the plugin homepage.

Direct download

Any comments and suggestions regarding the plugin are welcome.

[fenix_productions]

Bug which happens after activating FTP.

Steps to reproduce:
1. open DeliciousPosts plugin panel and log in into it;
2. wait for bookmarks to download;
3. browse them for a while;
4. go to your C:\ drive,
5. hit Ctrl+F and connect to some FTP;
6. close the connection using Disconnect button above files list.

Access violation message box appears:

Code: Select all

---------------------------
Total Commander
---------------------------
Access violation at address 00000000. Read of address 00000000.
Please report this error to the Author,
with a description of what you were doing when this error occurred!
Continue execution?

Windows exception: C0000005

Stack trace:

00000000  00413E4C  7E368732  7E368814

7E37A011  7E37A037  00442FDE  00413E4C

7E368732  7E368814  7E3689CB  7E37F409

7E379FB0  7E368A0E  004194FB  00419530

004196EA  00699394  7C817075  7C81707E


Press Ctrl+C to copy this report!
---------------------------
Tak   Nie   
---------------------------

Notes:
- sometimes this bug appear while connecting to FTP right away.
- it does not happen for TC 7.04a so I hope Christian will stop by here.

[fenix_productions]

I am writing second post just to separate from bug report.

Suggestions:
- make use of Disconnect button,
- allow to store details in more secure way - TC 7.50 makes built-in encryption use possible;

Is there really need for keeping your own SSL libraries in plugin folder? AFAIK they can be already available inside TC directory if someone uses them for secure connections. If not, use bandled DLLs then.

This part might be also the reason of previously mentioned error. I saw similar behaviour when I had reinstalled SSL on my system once in the past.

[HomoLogicus]

Bug which happens after activating FTP.
Steps to reproduce:

I have unsuccessfully tried to reproduce this bug several times Do you use the latest version of the plugin - 2.0.0.88? I have released the new version with TC 7.5 UNICODE features support recently. Maybe the bug has disappeared in this version?

Suggestions:
- make use of Disconnect button,
- allow to store details in more secure way - TC 7.50 makes built-in encryption use possible

These are good suggestions, thank you I will implement them in the next release.

Is there really need for keeping your own SSL libraries in plugin folder? AFAIK they can be already available inside TC directory if someone uses them for secure connections. If not, use bandled DLLs then.

How can they appear inside TC folder for example? I need to test my plugin with such libraries.

[Mikefield]

The SSL library is also used by the sftp plugin and the internal ftps. If the library is in TC main
directory (%Commander_Path%) it's not needed to update any plugin folder which uses the library.

mf

[fenix_productions]

2HomoLogicus
I am using the latest version of your plugin. As I previously mentioned: this might be related to SSL libraries. I have them all next to TOTALCMD.EXE file (for TC use only) and in DeliciousPosts folder. I assume they interrupt themselves somehow.

I will check it later though.

[HomoLogicus]

2Mikefield
If I understood correctly, I can test my plugin with the libraries from curl.haxx.se used by the SFTP plugin placed in TC main directory. Am I right? OK. But then my question is: What libraries must my plugin use if the libraries placed in the SFTP plugin directory but not in TC main directory?

[Mikefield]

If I understood correctly, I can test my plugin with the libraries from curl.haxx.se used by the SFTP plugin placed in TC main directory. Am I right?

Yes, you are.

OK. But then my question is: What libraries must my plugin use if the libraries placed in the SFTP plugin directory but not in TC main directory?

I know, it's not so easy, different points have to consider, there isn't just one way of doing it:
1. Not every user have the sftp plugin installed.
2. The SSL library is not delivered with the sftp plugin, every user have to install and maintain it by himself.
3. The plugin.inf is not designed to copy files in different folders (plugin and main folder).
4. Your plugin could not know if a sftp plugin exist and where the library is located.
5. You can use the ssl library in your plugin directory,
6. Your plugin should be able to use the library in the TC main folder.
7. If you don't use the plugins.inf and made your own install routine, it could check up the points above.
8. Also you could set in ini file where the librayry is located.

mf

[HomoLogicus]

The new version 2.1 is available for download.

What's new?

• Using SSL libraries from TC folder or SFTP plugin folder if they are available
• Keeping account information in the TC secure store if this feature is listed in the plugin settings (TC 7.5 and above only)
• Using the Disconnect button on FTP toolbar for quick logout

[HomoLogicus]

2fenix_productions

Did you try the new version? What about the problem with FTP connection? Is it OK with it now on your side?

TIA

[fenix_productions]

2HomoLogicus
Sorry. I was busy with few other things.

Now it works a lot better but there is still something wrong: although it does not crash right away on FTP, it crashes when closing TC.

First I have the message (roughly translated to English):

Code: Select all

TOTALCMD.exe - Application error
---------------------------
Instruction from "0x02383db2" calling memory address from "0x03a805b0". Memory cannot be "read".

Click OK to close application.
Click Cancel to start debugging.
---------------------------
OK   Cancel   
---------------------------

After OK smaller one appears:

Code: Select all

---------------------------
Error
---------------------------
Runtime error 216 at 08343DB2
---------------------------
OK   
---------------------------

And now suggestion time!

When I use Enter on DeliciousPosts and fail to provide a password for my account (by hitting Cancel), your plugin shows me Login and Settings items. Using the first one brings out the form for new account. Unfortunately I couldn't get TC password prompt again without restart so maybe giving additional item (next to mentioned two) to call it would be nice?

Mayke it simple: if user data exist in INI then there will be: Login, New Login and Settings. If there is nothing, show only two of them.

[HomoLogicus]

2fenix_productions

Now it works a lot better but there is still something wrong: although it does not crash right away on FTP, it crashes when closing TC.

Could you tell me a list of plugins installed with your TC? I can't reproduce this issue on my computer

if user data exist in INI then there will be: Login, New Login and Settings. If there is nothing, show only two of them.

I need to think about this suggestion

[fenix_productions]

2HomoLogicus
Here is the list:
http://fenixproductions.republika.pl/tc/report.zip

PS. If you are thinking about last suggestion then this is the first step to "multi-accounts" plugin

[HomoLogicus]

2fenix_productions

Here is the list

Thank you very much. You have a huge list of installed plugins! I'm shocked

if user data exist in INI then there will be: Login, New Login and Settings.

If you are thinking about last suggestion then this is the first step to "multi-accounts" plugin

Stop, stop, stop. What do you mean? What is the "New Login"? And what is the "multi-accounts plugin"? Delicious Posts can connect to different existing del.icio.us accounts. Login information for each account stores in TC secure area, but to access to the area a user need to enter the TC master password once. What's wrong with the logic?

[fenix_productions]

Stop, stop, stop. What do you mean? What is the "New Login"? And what is the "multi-accounts plugin"? Delicious Posts can connect to different existing del.icio.us accounts. Login information for each account stores in TC secure area, but to access to the area a user need to enter the TC master password once. What's wrong with the logic?

I am sorry I had to miss this part. If there is the possibility to have different accounts stored than it is OK.

Currently, the case when user typed wrong password or cancelled such operation, "Login" asks for new account details but I thought about little different scenario:
- Login - asks for master password again,
- New account - behaves like Login at this moment.

I understand that this might be just a matter of words so maybe keep Login as it is and add "Stored accounts"?