Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-39400

ServerConfigurationService should have more fallback defaults for some properties

    XMLWordPrintable

    Details

    • Type: Feature Request
    • Status: OPEN
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 11.5 [Tentative], 12.1, 19.0
    • Fix Version/s: None
    • Component/s: Kernel
    • Labels:
      None
    • Previous Issue Keys:
      KNL-1591
    • Test Plan:
      Hide

      Please add a Test Plan here.

      Show
      Please add a Test Plan here.

      Description

      In what I think would fix a number of bugs (will link them as related) it would be nice if ServerConfigurationService had more defaults for some properties.

      Some of these default can and are set by kernel.properties but some can't be set because they need a value that isn't defined when kernel.properties is read (it's read first). It would need to be set after all other properties are read then set IFF the property wasn't set by something else.

      Some variables that come up like this and are not always defined correctly are either defined as null or

      mail.support -> usually "support@"+ serverConfigurationService.getServerName()
      setup.request -> usually or "no-reply@" + serverConfigurationService.getServerName())

      This would result in a lot of nice code cleanup where this is defined in the code, either correctly, incorrectly or not at all.

      I believe additionally there should be at least an INFO message in the logs when one of these defaults is used recommending these be set.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  maintenanceteam Core Team
                  Reporter:
                  jonespm Matthew Jones
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                    Dates

                    Created:
                    Updated:

                      Git Integration