Details

      Description

      If possible, log a clear hint to a running security manager being a problem for JSPWiki, in order to avoid long and difficult problem analysis in users' environments.
      This should of course only be a transitional hack.

      1. JSPWIKI-705.patch
        0.8 kB
        Florian Holeczek

        Activity

        Hide
        florianh Florian Holeczek added a comment -

        changed summary

        Show
        florianh Florian Holeczek added a comment - changed summary
        Hide
        florianh Florian Holeczek added a comment -

        Also add a hint to the README.
        Does this affect 3.0, too?

        Show
        florianh Florian Holeczek added a comment - Also add a hint to the README. Does this affect 3.0, too?
        Hide
        florianh Florian Holeczek added a comment -

        Also see JSPWIKI-698. Should be done in 2.8.5.

        Show
        florianh Florian Holeczek added a comment - Also see JSPWIKI-698 . Should be done in 2.8.5.
        Hide
        florianh Florian Holeczek added a comment -

        maybe also warn in Install.jsp

        Show
        florianh Florian Holeczek added a comment - maybe also warn in Install.jsp
        Hide
        metskem Harry Metske added a comment -

        I built the check in SessionMonitor, this is the first piece of JSPWiki code that get's control (and immediately fails because it tries to init a logger, which fails if java security is active).
        See the committed changes, let me know if you want something else.

        (Doing something in Install.jsp makes no sense since the whole webapp (JSPWiki) does not get started.)

        Show
        metskem Harry Metske added a comment - I built the check in SessionMonitor, this is the first piece of JSPWiki code that get's control (and immediately fails because it tries to init a logger, which fails if java security is active). See the committed changes, let me know if you want something else. (Doing something in Install.jsp makes no sense since the whole webapp (JSPWiki) does not get started.)
        Hide
        florianh Florian Holeczek added a comment -

        Hi Harry,

        unfortunately you're right with the Install.jsp - but would have been extremely user friendly

        Regarding your commit, I had chosen a different approach (see the attached patch).
        The main difference is that it doesn't prevent the wiki from being started. This way, a workaround (full permissions for this webapp) may be applied without needing to recompile.

        WDYT?

        Show
        florianh Florian Holeczek added a comment - Hi Harry, unfortunately you're right with the Install.jsp - but would have been extremely user friendly Regarding your commit, I had chosen a different approach (see the attached patch). The main difference is that it doesn't prevent the wiki from being started. This way, a workaround (full permissions for this webapp) may be applied without needing to recompile. WDYT?
        Hide
        metskem Harry Metske added a comment -

        I'll have a look tonight, thanks.

        Show
        metskem Harry Metske added a comment - I'll have a look tonight, thanks.
        Hide
        metskem Harry Metske added a comment -

        Fixed in 2.8.5-svn-9 and 3.0.0-svn-232.

        Show
        metskem Harry Metske added a comment - Fixed in 2.8.5-svn-9 and 3.0.0-svn-232.
        Hide
        florianh Florian Holeczek added a comment -

        Updates the ReleaseNotes, too.

        Show
        florianh Florian Holeczek added a comment - Updates the ReleaseNotes, too.
        Hide
        florianh Florian Holeczek added a comment -

        Reopen in order to add fix version 2.9

        Show
        florianh Florian Holeczek added a comment - Reopen in order to add fix version 2.9
        Hide
        florianh Florian Holeczek added a comment -

        Closing this, since 2.9 has been released

        Show
        florianh Florian Holeczek added a comment - Closing this, since 2.9 has been released

          People

          • Assignee:
            Unassigned
            Reporter:
            florianh Florian Holeczek
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development