fcrepo-exts / fcrepo-vagrant

Fedora 4 Vagrant Virtual Machine

26 stars 22 forks source link

Header provider #92

Closed whikloj closed 5 years ago

whikloj commented 5 years ago

JIRA Ticket: n/a

DO NOT MERGE

This is based off of https://github.com/fcrepo4-exts/fcrepo4-vagrant/pull/91 and so should not be merged before it is merged and this is rebased.

What does this Pull Request do?

Installs a customized fcrepo-config.xml with the headerPrincipalProvider enabled.

How should this be tested?

Need to stand up this vagrant, then create a resource and restrict it to one of the users (ie. adminuser).

Then try to access that resource using the other user and the header ie.

curl -i -u testuser:password1 -H"some-header: adminuser" http://localhost:8080/fcrepo/rest/your_restricted_resource

Additional Notes:

Example:

Does this change require documentation to be updated? yes, some information about the header might be helpful.
Does this change add any new dependencies? no
Does this change require any other modifications to be made to the repository (ie. Regeneration activity, etc.)? no
Could this change impact execution of existing code? no

Interested parties

@fcrepo4/committers

dbernstein commented 5 years ago

@whikloj : I have rebased this PR on my previous PR (https://github.com/fcrepo4-exts/fcrepo4-vagrant/pull/94 (which @yinlinchen is testing). Once that one is merged, I will merge the rebased version of this PR which you'll find here: https://github.com/fcrepo4-exts/fcrepo4-vagrant/pull/95