Uploaded image for project: 'Spring Web Services'
  1. Spring Web Services
  2. SWS-551

Support @Secured for @PayloadRoot methods

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Cannot Reproduce
    • Affects Version/s: 1.5.7
    • Fix Version/s: 2.0.1
    • Component/s: Core
    • Labels:
      None

      Description

      Currently when message dispatcher tries to invoke method of an @Endpoint, where method has been @Secured using Spring-AOP JDK dynamic proxy, IllegalArgumentException gets thrown (from MethodEndpoint's invoke) with message "object is not an instance of declaring class". Guess it has to do with fact that method is actually being invoked on a proxy implementing same interface as endpoint whilst it seems that instance of endpoint is expected. Haven't tried yet but I think that if CGLIB proxy was used that this should already work.

        Issue Links

          Activity

          Hide
          sdomsta Shaun Domingo added a comment -

          Hi Arjen,

          Is there any way you can give any guidance/help around how to get around this problem, or if/when this will be fixed?

          We really appreciate all the hard work that goes into this project.

          Cheers,
          Shaun

          Show
          sdomsta Shaun Domingo added a comment - Hi Arjen, Is there any way you can give any guidance/help around how to get around this problem, or if/when this will be fixed? We really appreciate all the hard work that goes into this project. Cheers, Shaun
          Hide
          arjen.poutsma Arjen Poutsma added a comment -

          Shaun, we'll take a look at this for 2.0.

          Show
          arjen.poutsma Arjen Poutsma added a comment - Shaun, we'll take a look at this for 2.0.
          Hide
          arjen.poutsma Arjen Poutsma added a comment -

          With the new @Endpoint programming model in 2.0, I can't seem to reproduce this.

          If it still doesn't work for you, feel free to reopen and attach a testcase which illustrates the problem.

          Show
          arjen.poutsma Arjen Poutsma added a comment - With the new @Endpoint programming model in 2.0, I can't seem to reproduce this. If it still doesn't work for you, feel free to reopen and attach a testcase which illustrates the problem.
          Hide
          arjen.poutsma Arjen Poutsma added a comment -

          Closing old issues

          Show
          arjen.poutsma Arjen Poutsma added a comment - Closing old issues

            People

            • Assignee:
              arjen.poutsma Arjen Poutsma
              Reporter:
              sslavic Stevo Slavić
            • Votes:
              5 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: