Skip to content
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

Cleaning up more FilePath API restrictions #8924

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

rsandell
Copy link
Member

@rsandell rsandell commented Feb 1, 2024

This is a follow up to #8913
As these methods looks to also be leftovers from an old security release.

Proposed changelog entries

  • Developer: Grant access to more FilePath APIs.

Proposed upgrade guidelines

N/A

Submitter checklist

Desired reviewers

@mention

Before the changes are marked as ready-for-merge:

Maintainer checklist

@rsandell rsandell added the developer Changes which impact plugin developers label Feb 1, 2024
@rsandell rsandell requested review from Wadeck, timja, a team and NotMyFault February 1, 2024 12:41
@rsandell
Copy link
Member Author

rsandell commented Feb 5, 2024

@NotMyFault could you please take a look at this one as well, it would be good if it was part of the same release as #8913

Copy link
Member

@NotMyFault NotMyFault left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NotMyFault could you please take a look at this one as well, it would be good if it was part of the same release as #8913

Surely :)

@timja
Copy link
Member

timja commented Feb 5, 2024

/label ready-for-merge


This PR is now ready for merge, after ~24 hours, we will merge it if there's no negative feedback.

Thanks!

@comment-ops-bot comment-ops-bot bot added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label Feb 5, 2024
@NotMyFault NotMyFault merged commit b91a57c into jenkinsci:master Feb 6, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer Changes which impact plugin developers ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants