Uploaded image for project: 'Magnolia REST Framework'
  1. Magnolia REST Framework
  2. MGNLREST-73

When one endpoint definition fails to register, the rest of the to-be-registered endpoint will not get registered.

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Blocker
    • 1.2.1
    • 1.1.1
    • None
    • None
    • Basel 86
    • 3

    Description

      In the RestDispatcherServlet#init (line 112), all currently registered endpoints are being registered to the servlet. During this process, the Endpoint will be initialized. We had a scenario were one of the endpoint could not be initialized properly due to a wrong Generic type. The initialization process did throw a guice error without any real description of the failure, but this was not really the problem.
      The servlet threw an exception and did not register the rest of the valid endpoint keeping us in the dark about what in which endpoint did do wrong.

      I would like to suggest a try..catch around the line "registerEndpoint(endpointDefinition)" so developers know which Endpoint definition failed to register, and so that the rest of the endpoint are correctly registered (because they are fine).

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              ilgun Ilgun Ilgun
              jdiepeveen Jordie Diepeveen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD