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

story 72982948. Normal user can only change properties but not add child nodes. #626

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@mohideen
Contributor

mohideen commented Nov 8, 2014

mohideen added some commits Nov 7, 2014

story 72982948. Normal user can only change properties but not add ch…
…ild nodes.

Fix to only check ancestor existence instead of getting each ancestor that might raise AccessDeniedException if the user lacks permission to an ancestor.
Updated test cases.

https://www.pivotaltracker.com/story/show/72982948

@peichman-umd peichman-umd deleted the umd-lib:ancestor-authorization-fix branch Dec 10, 2014

@bseeger

This comment has been minimized.

Member

bseeger commented Nov 6, 2015

Hi @mohideen, @peichman-umd and @awoods,
We are cleaning up the fcrepo4 repository on github. It looks like the umd-lib:ancestor-authorization-fix branch that this PR is requested from has been deleted. Is this pull request relevant anymore? If not, could you please close it? If it is still relevant, please create a issue in jira and rebase your branch to eliminate merge conflicts.

If we don't hear from you by November 20th, 2015 we will close it.

Thank you!

@awoods

This comment has been minimized.

Member

awoods commented Nov 9, 2015

Closed due to age of PR.

@awoods awoods closed this Nov 9, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment