Merge pull request #29 from NYULibraries/feature/omniauthhelper_rehash

Feature/omniauthhelper rehash
NS Ldap factory extra attrs should just be a hash no a new Entry
Reworked the provider mappers to use Forwardable
Loading properties after sub class inst vars are set
Re-kajigger the properties inst var inheritance
Use @omniauth_hash inst var, not local param
Application controller has to umbrella catch Mapper arg errors , not validator as it was before
Clean up raise errors
Removed rescue_from for Mapper::ArgumentError. It's unreachable because the validator class will always short circuit it
Merge pull request #31 from NYULibraries/feature/newschool_ldap_attributes

Feature/newschool ldap attributes
Merge branch 'development' of github.com:NYULibraries/login into feature/shibboleth-api