-
-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
Mark TarOutputStream as restricted #4272
Mark TarOutputStream as restricted #4272
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, unfortunately we still distribute some plugins outside the jenkinsci org from several years ago. For those, https://github.com/jenkins-infra/deprecated-usage-in-plugins works better as it downloads all currently distributed plugins and searches for uses of classes from a list of names. |
Putting |
Ok, that is a good idea. I also added this annotation to the TarInputStream and already created PRs for plugins which still use this class: |
Probably the most pragmatic approach.
|
+1 for |
I added the @RestrictedSince("2.200") as suggested |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I plan to merge it tomorrow if no negative feedback |
I'm pretty sure that it is not used anymore, because the search just shows no usage at all:
https://github.com/search?q=org%3Ajenkinsci+TarOutputStream&type=Code
Proposed changelog entries
Submitter checklist