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

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

Closed
wants to merge 2 commits into from
Closed

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

wants to merge 2 commits into from

Conversation

mohideen
Copy link
Contributor

@mohideen mohideen commented Nov 8, 2014

…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 ancestor-authorization-fix branch December 10, 2014 21:47
@bseeger
Copy link
Contributor

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
Copy link

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants