Uploaded image for project: 'OpenNMS'
  1. OpenNMS
  2. NMS-9133

Fix CORS section to include /api/* besides /rest

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: Meridian-2016.1.4, 18.0.4
    • Fix Version/s: 20.0.2, Meridian-2017.1.0
    • Component/s: REST
    • Security Level: Default (Default Security Scheme)
    • Labels:
      None
    • Sprint:
      Horizon - July 12th

      Description

      When implementing external applications that relies on the ReST API, CORS should be enabled on web.xml.

      Currently it has the following:

        <filter-mapping>
          <filter-name>CORS Filter</filter-name>
          <url-pattern>/rest/*</url-pattern>
        </filter-mapping>
        <filter-mapping>
          <filter-name>CORS Filter</filter-name>
          <url-pattern>/nrt/*</url-pattern>
        </filter-mapping>
      

      It should be:

        <filter-mapping>
          <filter-name>CORS Filter</filter-name>
          <url-pattern>/api/*</url-pattern>
        </filter-mapping>
        <filter-mapping>
          <filter-name>CORS Filter</filter-name>
          <url-pattern>/rest/*</url-pattern>
        </filter-mapping>
        <filter-mapping>
          <filter-name>CORS Filter</filter-name>
          <url-pattern>/nrt/*</url-pattern>
        </filter-mapping>
      

        Attachments

          Activity

            People

            • Assignee:
              seth Seth Leger
              Reporter:
              agalue Alejandro Galue
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: