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

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

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Trivial
    • Resolution: Fixed
    • Meridian-2016.1.4, 18.0.4
    • 20.0.2, Meridian-2017.1.0
    • REST
    • Security Level: Default (Default Security Scheme)
    • None
    • 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

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

            Dates

              Created:
              Updated:
              Resolved: