Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/gerrit-code-review-plugin
  Commit: f88f8341f50eb3dbb843cc92538a0aa481314c53
      
https://github.com/jenkinsci/gerrit-code-review-plugin/commit/f88f8341f50eb3dbb843cc92538a0aa481314c53
  Author: Zhao Xiaojie <linuxsu...@gmail.com>
  Date:   2019-04-11 (Thu, 11 Apr 2019)

  Changed paths:
    M README.md

  Log Message:
  -----------
  Typo fixes


  Commit: e6c56b7cda00d54a31bdc74117dcbd5013fc2612
      
https://github.com/jenkinsci/gerrit-code-review-plugin/commit/e6c56b7cda00d54a31bdc74117dcbd5013fc2612
  Author: Jon Sten <j...@axis.com>
  Date:   2019-06-13 (Thu, 13 Jun 2019)

  Changed paths:
    M README.md
    M src/main/java/jenkins/plugins/gerrit/GerritWebHook.java

  Log Message:
  -----------
  Running web hook as System ACL

For locked down Jenkins instances this is a must. In our case anonymous
doesn't have overall read access and this nothing happens when gerrit
pushes to the notification URL. This change changes so that the system
ACL is used during resolution of which jobs to trigger.

As with every privilege escalation one should ask oneself if it is safe
and necessary. In this case I would say that it is necessary, since it
makes it possible to use this plugin in locked down enterprise
situations. The change should also be safe, during privilege
escalation noting is written to the caller, which means that no
information about job names or folder setup can be leaked due to this
change. Additionally the only place where the callers input is used, is
during check if the remote URL equals the job remote URL, and that is
done using regular string operations. This should lead to a very small
attack surface.

Change-Id: Icac60435abd77ff462f72cfc1dbe831c768c8a90


Compare: 
https://github.com/jenkinsci/gerrit-code-review-plugin/compare/e626a37c6885...e6c56b7cda00

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-commits+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/gerrit-code-review-plugin/push/refs/heads/master/e626a3-e6c56b%40github.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to