Uploaded image for project: 'Jetspeed (Retired)'
  1. Jetspeed (Retired)
  2. JS1-523

mail.server setting in JetspeedResources.properties is ignored

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 1.6-dev
    • None
    • Turbine
    • None
    • J2SDK 1.4.2, Tomcat 5.0.27, Linux Redhat/Windows 2000

    Description

      The key "mail.server" setting in JetspeedResources.properties is ignored by Turbine when sending confirmation emails to users. This is caused by the following code in SendConfirmationEmail.java (among others):

      Properties props = System.getProperties();
      String mailServerMachine = JetspeedResources.getString( "mail.server" );
      props.put ( "mail.host", mailServerMachine );
      props.put("mail.smtp.host", mailServerMachine);

      se.send();

      Jetspeed uses the key "mail.host" whereas Turbine actually uses "mail.server" as the key when sending mails. As a result, the smtp server used when sending mails is always "localhost" (the default in Turbine) and not the smtp server set in JetspeedResources.properties.

      To reproduce:
      1. Run Jetspeed in a box with no SMTP server (or a server behind a firewall).
      2. Set the key "mail.server" in JetspeedResources.properties to your SMTP server (which is different from the server running Jetspeed)
      3. Set newuser.confirm.enable=true, newuser.confirm.email.from=portal-admin@yourhost.com
      4. Try creating a new account via the top navigation bar. Set the email of the new user to an address outside your network. Use the admin account to enable the new user.

      Wait for sometime, either the confirmation email bounces back to portal-admin@yourhost.com or the user never receives the confirmation email at all.

      To resolve:
      Change "mail.host" to "mail.server" in the following files: JLoginUser.java, SendConfirmationEmail.java, SendNewUserNotificationEmail.java, UserUpdateAction.java

      Attachments

        Activity

          People

            Unassigned Unassigned
            jaqmarit Jaq Marit
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated: