[jboss-dev-forums] [Design of JBoss ESB] - Re: RemoteGatewayListener, RemoteFileSystemFactory, RemoteFi

[Kevin.Conner@jboss.com]

"tfennelly" wrote : This code is very interesting indeed :)
I suspect it is no different from the rest of the code ;)

"tfennelly" wrote : These settings are used by the FtpImpl and SecureFtpImpl classes, which are RemoteFileSystem implementations.  The are constructed from an EPR by the RemoteFileSystemFactory.  Both implementations initialise their ftpLocalDir and ftpRemoteDir config settings from the jboessesb-properties.xml.
I'll be very surprised if this is the case, are you certain of this?  How could any of the ftp/sftp stuff work in that case.  As already mentioned on the JIRA issue, it looks like the local dir is not getting passed through but the remote must be getting through otherwise it just wouldn't work.

As for the local dir, I'm not sure this should even be in the EPR to be honest as it should be specific to the local system.  It would be better if this was being taken from jbossesb-properties.xml and defaulting to the value of java.io.tmpdir.

"tfennelly" wrote : It then loads this into a ConfigTree instance, calls checkParams, which goes and initializes the ftpLocalDir and ftpRemoteDir variables again from the configtree that was just constructed from the config.  Huh?
Huh as in you are surprised or huh as in it appears to be doing it in a similar way to the others?

"tfennelly" wrote : I hope somebody sees the logic in all of this toing and frowing with the config!!
Not really, but it seems to be consistent with the rest of it.  But then you know this as well as everyone else :)

"tfennelly" wrote : Anyway... does anyone know what the real intention is here?  ftpLocalDir and ftpRemoteDir are currently being configured from the jbossesb-properties.xml.
Remote must be coming from the config, surely, although local doesn't really make sense.
"tfennelly" wrote : Was there ever an intention to allow this to be overridden by the EPR ala how it is currently being set in the configtree passed by the RemoteGatewayListener to the ListenerUtil.assembleEpr method?
The remote definitely, and I'm pretty sure this already works.  The local doesn't make any sense to me :)

"tfennelly" wrote : As I see it, this code in RemoteGatewayListener is currently doing nothing.  Can we just remove it and skip the EPR override that we don't have?
Which code are you referring to?  Can you give line numbers?

"tfennelly" wrote : Also on this FTP code... looks to me like RemoteFileSystem instances are potentially being created for fun by (e.g.) the RemoteFileMessageComposer i.e. for every message.  Is this really necessary, or can we just create one for the RemoteFileMessageComposer instance?
I'm sure there are many improvements to this code, like the others.  I suspect the reason for this is more to do with the underlying connections and the lack of recovery in the original donation rather than anything else but file a JIRA for it.

Lets face it, the original code did not even understand how to construct a proper ftp URL, leaving us in the position where we can't easily fix it without breaking everything out there.

Kev

View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4240625#4240625

Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=4240625