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

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

    XMLWordPrintable

    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 (Inactive)
            Reporter:
            agalue Alejandro Galue
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Git Integration