jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jukka Zitting (JIRA)" <j...@apache.org>
Subject [jira] Commented: (JCRRMI-13) Enhancement to get RMI through firewalls
Date Mon, 06 Apr 2009 22:08:12 GMT

    [ https://issues.apache.org/jira/browse/JCRRMI-13?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12696276#action_12696276

Jukka Zitting commented on JCRRMI-13:

bq. Do we really need support for the system property ?

The system property does come in quite handy in cases like jackrabbit-standalone or other
custom deployments, where we then won't need to explicitly add some other configuration mechanism.
I don't see much harm in keeping the functionality, as it only affects behaviour when you
have explicitly set the system property. And if you're paranoid, you can always explicitly
disable the effect of the system property by calling setPortNumber(0).

bq. Can we also have the portnumber as an argument to a new constructor instead of just a
setter (setter is still good, of course) ?

I prefer the setter approach as it automatically works with any ServerAdapterFactory subclasses
(like JackrabbitServerAdapterFactory), but I wouldn't mind if we also added extra constructors.

> Enhancement to get RMI through firewalls
> ----------------------------------------
>                 Key: JCRRMI-13
>                 URL: https://issues.apache.org/jira/browse/JCRRMI-13
>             Project: Jackrabbit JCR-RMI
>          Issue Type: Improvement
>         Environment: RHEL Linux 5.0
>            Reporter: Tony Richardson
>            Assignee: Jukka Zitting
>             Fix For: 2.0
> It is difficult to get RMI through a firewall with the current implementation of org.apache.jackrabbit.rmi.server.ServerObject.
As it selects a random port for RMI execution. This issue can be resolved by adding a system
property and modifying the default constructor as shown below.
>   private static Integer bindPort = Integer.getInteger("org.apache.jackrabbit.rmi.port",
new Integer(0));
>     /**
>      * Creates a basic server adapter that uses the given factory
>      * to create new adapters.
>      *
>      * @param factory remote adapter factory
>      * @throws RemoteException on RMI errors
>      */
>     protected ServerObject(RemoteAdapterFactory factory)
>             throws RemoteException {
>         super(bindPort.intValue());
>         this.factory = factory;
>     }

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message