Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-35493

severe performance regression after SECURITY-243

      If you have a git changelog with lots of commits by different users and a non-local authentication scheme (basically anything other than the local database) then viewing that page now takes a lot longer as all the users in the commits need to be looked up in the security realm to see if they are valid authentication "users" or if it is just a "full name" that can be resolved from disk.

      There needs to be a way for plugins to say get me a user not for authentication purposes that can if the user has been saved on disk will return that in preference to not hitting the security realm to see if the user does indeed exist.

      setting hudson.model.User.SECURITY_243_FULL_DEFENSE=false helps but there should really be a separate API.

      WIthout this extra API all security realms need to implement multiple caches (a not found cache as well as a regular cache)

          [JENKINS-35493] severe performance regression after SECURITY-243

          James Nord created issue -
          James Nord made changes -
          Component/s New: core [ 15593 ]
          Component/s Original: core [ 15738 ]
          Key Original: SECURITY-311 New: JENKINS-35493
          Workflow Original: Security v1.2 [ 171796 ] New: JNJira [ 171796 ]
          Project Original: Security Issues [ 10180 ] New: Jenkins [ 10172 ]
          Status Original: Untriaged [ 10001 ] New: Open [ 1 ]
          Jesse Glick made changes -
          Labels Original: regression New: performance regression
          Jesse Glick made changes -
          Link New: This issue is blocking SECURITY-243 [ SECURITY-243 ]
          James Nord made changes -
          Link New: This issue is related to JENKINS-35484 [ JENKINS-35484 ]
          rsandell made changes -
          Assignee New: rsandell [ rsandell ]
          rsandell made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          rsandell made changes -
          Remote Link New: This issue links to "PR 2446 (Web Link)" [ 14635 ]
          rsandell made changes -
          Labels Original: performance regression New: lts-candidate performance regression
          Spike Washburn made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 171797 ] New: JNJira + In-Review [ 199165 ]

            rsandell rsandell
            teilo James Nord
            Votes:
            1 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: