Uploaded image for project: 'Apache NiFi'
  1. Apache NiFi
  2. NIFI-3862

ListenHTTPServlet should set the issuerDN as well

    Details

    • Type: Improvement
    • Status: Patch Available
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 0.7.1
    • Fix Version/s: None
    • Component/s: Core Framework
    • Labels:
    • Flags:
      Patch

      Description

      The ListenHtttpServlet currently sets the remote user DN as an attribute in the flowfile, but it really should set the remote issuer dn as well. This will allow us to verify that the remote host actually as permission to hit our server using those two pieces of information.

        Activity

        Hide
        chentecaliente Vincent Russell added a comment -

        submitting a patch to provide issuerDN in FlowFile

        Show
        chentecaliente Vincent Russell added a comment - submitting a patch to provide issuerDN in FlowFile
        Hide
        chentecaliente Vincent Russell added a comment -

        putting issuerDN in ListenHttpServlet

        Show
        chentecaliente Vincent Russell added a comment - putting issuerDN in ListenHttpServlet
        Hide
        joewitt Joseph Witt added a comment -

        Vincent Russell

        Can you confirm this is built for the 0.x line instead of the 1.x line? We'd want this to be made for the 1.x line at least and then optionally considered for 0.x inclusion. The fix version being 0.7.3 is probably not correct so going to remove that. Once this is near merge we can adjust the fix version to the appropriate next 1.x release line.

        You changed the OnStopped annotation to OnShutdown. Was that important for this improvement? If not I'd recommend avoiding that change.

        Show
        joewitt Joseph Witt added a comment - Vincent Russell Can you confirm this is built for the 0.x line instead of the 1.x line? We'd want this to be made for the 1.x line at least and then optionally considered for 0.x inclusion. The fix version being 0.7.3 is probably not correct so going to remove that. Once this is near merge we can adjust the fix version to the appropriate next 1.x release line. You changed the OnStopped annotation to OnShutdown. Was that important for this improvement? If not I'd recommend avoiding that change.

          People

          • Assignee:
            Unassigned
            Reporter:
            chentecaliente Vincent Russell
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development