Telnet Server Library Icon


User Ratings

ease 1 of 5 2 of 5 3 of 5 4 of 5 5 of 5 4 / 5
features 1 of 5 2 of 5 3 of 5 4 of 5 5 of 5 4 / 5
design 1 of 5 2 of 5 3 of 5 4 of 5 5 of 5 3 / 5
support 1 of 5 2 of 5 3 of 5 4 of 5 5 of 5 0 / 5
Write a Review

User Reviews

  • 1 of 5 2 of 5 3 of 5 4 of 5 5 of 5

    The way it has been designed is quite interesting. Like keeping annotations on commands and then adding them to handler. Its a nice way to allow others add more stuff to it... However, i found something not correct. The server passing its instance to all the telnet clinet threads which call the servers processCommand method for execution of the command is not correct. Firstly It creates a cyclic loop on the code. Like Server dependent on client and client dependent on the server. Secondly, there is no synchronization on the processCommand method and hence i am sure, it will kind of mess up things, when multiple clients try out stuff at the same time. I would have kept the processCommand code in the client classes, rather than in the server, or else provided some sort of synchronization. But, keep in the client class makes it more fruitful, i suppose. And again, i am really not sure, what is the purpose of the javaLogger in the whole project. I think, i will take references from your code and implement a new one. Adding commands through annotations is a very nice idea, as far as the easiness to extend the code is concerned. But, using reflections makes it again a little less performant. Thanks.

    Posted 07/02/2016

Thanks for helping keep SourceForge clean.

Screenshot instructions:
Red Hat Linux   Ubuntu

Click URL instructions:
Right-click on ad, choose "Copy Link", then paste here →
(This may not be possible with some types of ads)

More information about our ad policies

Briefly describe the problem (required):

Upload screenshot of ad (required):
Select a file, or drag & drop file here.

Please provide the ad click URL, if possible:

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.

No, thanks