New AccessHandler Module

Bug #1413624 reported by Thomas Berezansky on 2015-01-22
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Wishlist
Unassigned

Bug Description

The branch below contains a new AccessHandler module intended for a more patron-friendly (OPAC auth instead of basic auth) protection for content and Apache features.

I have included documentation on use of the module, including a quick "how to use it to protect an Apache proxy" example, in both "Release Notes" form and in TechRef. The two copies should currently be identical other than heading levels. I would not object to the Release Notes version being reduced (perhaps pulling the example proxy config out entirely?).

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/AccessHandler

Mike Rylander (mrylander) wrote :

This looks good, though I haven't tested it.

My only comment at this point is that there is a pre-existing pattern to use space-separated lists, instead of comma-separated, in O::W::Proxy and ::Proxy::Authen. I think it would be preferable to follow that pattern to reduce admin confusion.

Thanks, Thomas.

Thomas Berezansky (tsbere) wrote :

Thanks for the comment, I have pushed a bugfix that includes allowing commas and spaces on the lists, allowing admins to use either (or both, really).

Changed in evergreen:
milestone: none → 2.8-beta
Galen Charlton (gmc) on 2015-02-19
Changed in evergreen:
importance: Undecided → Wishlist
assignee: nobody → Galen Charlton (gmc)
Galen Charlton (gmc) wrote :

I've run it through its pace and looks good to me, so I've pushed it to master, along with a minor tweak of the documentation to reflect the change allowing space-delimited option lists.

Thanks, Thomas!

Changed in evergreen:
status: New → Fix Committed
assignee: Galen Charlton (gmc) → nobody
Remington Steed (rjs7) wrote :

I pushed a commit that moves the docs from TechRef into the main docs folder and incorporates them into the "System Configuration and Customization" section of the docs. Thanks Thomas!

Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers