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

Raise exception when access is denied instead of silently failing #257

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

bluesliverx
Copy link
Contributor

@bluesliverx bluesliverx commented Sep 30, 2022

This fixes #163 in the way asked for by @jetersen, but also adds a unit test to ensure correct behavior. I chose not to base it on the failIfNotFound flag since access denied shows a configuration issue that should be addressed immediately instead of "I have permissions, but it just doesn't exist" that failIfNotFound is for.

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@bluesliverx
Copy link
Contributor Author

I didn't see an existing issue except for the PR in #163, but we've run into this recently and it caused some serious issues and was VERY surprising to me as I just thought things were working, but it was actually silently failing.

@bluesliverx
Copy link
Contributor Author

RETEST

@niclan
Copy link

niclan commented Aug 29, 2023

Dear maintainers, this needs to be merged (unless you have a different change in place already)

@bluesliverx
Copy link
Contributor Author

@jetersen I'm not sure if there are other maintainers here, but this PR has been open for nearly a year now. Any chance it could get a review/merge? We've been using it in our local fork with no issues in the last year.

@forest-code42
Copy link

I have raised a Jenkins JIRA ticket to please merge this PR https://issues.jenkins.io/browse/JENKINS-72324

@jetersen jetersen enabled auto-merge (squash) November 14, 2023 22:32
@jetersen jetersen added the bug label Nov 14, 2023
@jetersen jetersen merged commit 8dfe406 into jenkinsci:master Nov 14, 2023
13 of 14 checks passed
@jetersen
Copy link
Member

released in 362.v8dfe4061f29e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants