298930 - Create JSS client/server test program to test secure file transfer

Status: RESOLVED FIXED

(JSS Graveyard :: Tests, defect)

Description

[Sandeep Konchady]

Older release of JSS had an issue with gradually slowing socket read time when
transfering large files (> 10MB).  In order to test this in current and future
release, there needs to be a test client that can read a file and transfer it to
a server (remote or local) via JSS socket.  The server should report the number
of bytes read and the time it took to read these bytes.  There should not no
degradation in read time if there is no leak of any sort.

Comment 1

[Sandeep Konchady]

Attachment: JSS_FileUploadClient and JSS_FileUploadServer program to test secure file transfer using JSS

This is not part of all.pl, but is a client/server that uses JSS to transfer
files securely.  The main purpose of this test would be to test the performance
of large file transfer using JSS.

Comment 2

[glen beasley]

Comment on attachment 189245 [details] [diff] [review]
JSS_FileUploadClient and JSS_FileUploadServer program to test secure file transfer using JSS

you should put a
System.exit(0) at the 
exits of the the two programs, and also change 
error case where you call
System.exit to a nonzero
status code to indicate abnormal termination. 

Also, provide a note in this bug on your findings 
regarding BufferedStream.mark(Interger.MaxValue)
you might even want to put a comment in the code  to warn others.

Comment 3

[Sandeep Konchady]

Attachment: Modified JSS_FileUploadServer.java and JSS_FileUploadClient.java with critical notes

Updated the source with Glen's suggestions to place System.exit() calls
appropriately.

NOTE: If bufferedStream.mark(Integer.MAX_VALUE); method is invoked then fill
method of BufferedInputStream class copies lot of data using System.arraycopy
(which in-turn use memcpy). This causes very high CPU usage.  This is one of
the reasons secure large file transfer can become slow over time.

Comment 4

[glen beasley]

Comment on attachment 192301 [details] [diff] [review]
Modified JSS_FileUploadServer.java and JSS_FileUploadClient.java with critical notes

you could add:
For JSS_FileUploadServer 
usage last parameter [testInetAddress: true|false] instead
of just [true|false]

For the client you could 
pass in the file name
rather than 
" Default file to upload is foo.in located locally"

Comment 5

[Sandeep Konchady]

Commited the files JSS_FileUploadServer.java and JSS_FileUploadClient.java with
changes suggested by Glen.

Comment 6

[Wan-Teh Chang]

Could you set the target milestone?  Thanks.