• Icon: Improvement Improvement
    • Resolution: Done
    • Icon: Minor Minor
    • core
    • None

      ACL.impersonate uses pre-java 7 patterns and either ends up with requiring a finally or requiring additional classloading.

      Much more efficient if we can do something like

      try (ACLContext _ = ACL.as(auth)) {
        // do some stuff as auth
      }
      

          [JENKINS-36494] Use try with resources for impersonation

          Stephen Connolly created issue -
          Stephen Connolly made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Stephen Connolly made changes -
          Remote Link New: This issue links to "PR 2437 (Web Link)" [ 14591 ]
          Stephen Connolly made changes -
          Resolution New: Done [ 10000 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 173205 ] New: JNJira + In-Review [ 199415 ]
          Stephen Connolly made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]

            stephenconnolly Stephen Connolly
            stephenconnolly Stephen Connolly
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: