Uploaded image for project: 'Seam Faces'
  1. Seam Faces
  2. SEAMFACES-210

ViewConfigExtension should throw RuntimeException if having Security Annotation but no Security extension

        Details

        • Type: Feature Request
        • Status: Resolved (View Workflow)
        • Priority: Critical
        • Resolution: Done
        • Affects Version/s: None
        • Fix Version/s: 3.1.0.Beta5
        • Component/s: None
        • Labels:
          None

          Description

          ViewConfigExtension should throw RuntimeException if having Security Annotation but no Security extension. Otherwise users think they are security but are unsecured.

            Gliffy Diagrams

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              gastaldi George Gastaldi added a comment -

              I think that logging a WARN message should be enough. There may be use cases where it is impossible to remove the security annotations and throwing a runtime exception on it may be too harsh. WDYT ?

              Show
              gastaldi George Gastaldi added a comment - I think that logging a WARN message should be enough. There may be use cases where it is impossible to remove the security annotations and throwing a runtime exception on it may be too harsh. WDYT ?
              Hide
              Permalink
              clerum Cody Lerum added a comment -

              The Intent is to protect the user from misconfiguration or framework error. Log messages can be easily missed. I vote for halting deployment if possible.

              Show
              clerum Cody Lerum added a comment - The Intent is to protect the user from misconfiguration or framework error. Log messages can be easily missed. I vote for halting deployment if possible.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  clerum Cody Lerum
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development