Uploaded image for project: 'Struts 1'
  1. Struts 1
  2. STR-1817

Secure html:link tag, plus module support.

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Duplicate
    • Affects Version/s: Nightly Build
    • Fix Version/s: None
    • Component/s: Tag Libraries
    • Labels:
      None
    • Environment:
      Operating System: other
      Platform: Other
    • Bugzilla Id:
      24235

      Description

      I'd like to add a module="" attribute to the html:link and html:rewrite tags so
      that you can specify a different module to link to. In addition, I'd like to
      enhance the link tag to provide a security mechanism. The idea being that if
      you supply the action="" and/or module="" then the action mapping roles would
      be checked. If the user was NOT in the required role, the link would not be
      written.

      There are 2 options for the security feature. Creating a SecureLinkTag,
      html:slink, that will provide the security mechanism by default. The 2nd,
      would be to supply a secure="true/false" attribute to the html:link tag.

      I will submit patches to the latest nightly build for consideration. Thank
      you.

        Attachments

        1. ASF.LICENSE.NOT.GRANTED--TagUtils.patch
          8 kB
          Gary Ashley
        2. ASF.LICENSE.NOT.GRANTED--struts-html.patch
          26 kB
          Gary Ashley
        3. ASF.LICENSE.NOT.GRANTED--SecureLinkTag.patch
          5 kB
          Gary Ashley
        4. ASF.LICENSE.NOT.GRANTED--RewriteTag.patch
          0.5 kB
          Gary Ashley
        5. ASF.LICENSE.NOT.GRANTED--patch-examples.txt
          8 kB
          Gary Ashley
        6. ASF.LICENSE.NOT.GRANTED--patch3.txt
          39 kB
          Gary Ashley
        7. ASF.LICENSE.NOT.GRANTED--patch.txt
          25 kB
          Gary Ashley
        8. ASF.LICENSE.NOT.GRANTED--LinkTag.patch
          1 kB
          Gary Ashley

          Activity

            People

            • Assignee:
              husted Ted Husted
              Reporter:
              garyashley@toughguy.net Gary Ashley
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: