Evergreen

New AccessHandler Module

Bug #1413624 reported by Thomas Berezansky
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned
Evergreen 2.8-beta

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

Revision history for this message
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.

Revision history for this message
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).

Thomas Berezansky (tsbere)
Changed in evergreen:
milestone: none → 2.8-beta
Galen Charlton (gmc)
Changed in evergreen:
importance: Undecided → Wishlist
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
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
Revision history for this message
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!

Evergreen Bug Maintenance (bugmaster)
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.