-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
upgrade to commons-digester3 #4
base: master
Are you sure you want to change the base?
Conversation
fd3eeb3
to
cb77d0f
Compare
I am not a maintainer of this plugin. The code added here is the one suggested in core PR. So +1 on merging this. |
I don't think the official maintainer of the plugin is active anymore on GitHub. |
It seems that previous MR #2 was merged by @daniel-beck. Another MR #1 was merged by @olivergondza . Both seems to be active on GitHub. |
Both were merged in our (then) project-wide roles as security officer and infra team member, respectively. |
upgrade to commons-digester3. Follow up of jenkinsci/jenkins#5320
ping @daniel-beck