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.

Sign up for GitHub

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

Jump to bottom

Comment out LDAP settings by Default #3

Merged
merged 1 commit into from
Aug 7, 2013
Merged

Comment out LDAP settings by Default #3

merged 1 commit into from
Aug 7, 2013

Conversation

misilot
Copy link
Contributor

@misilot misilot commented Aug 6, 2013

Since all the other configuration options are commented by default, shouldn't LDAP be as well?

demiankatz added a commit that referenced this pull request Aug 7, 2013
@demiankatz
Merge pull request #3 from misilot/fixdefault-config-ldap
a951bc7 
Comment out LDAP settings by Default
@demiankatz demiankatz merged commit a951bc7 into vufind-org:master Aug 7, 2013
@misilot misilot deleted the fixdefault-config-ldap branch August 7, 2013 13:04
@demiankatz demiankatz mentioned this pull request Jan 29, 2014
Closed
@DCL-jbannon DCL-jbannon mentioned this pull request Apr 4, 2014
Closed
demiankatz referenced this pull request in crhallberg/vufind May 28, 2014
@demiankatz
Merge pull request #3 from crhallberg/ebsco
16e0af2 
Bootstrap 2 and jQuery Mobile theming
EreMaijala pushed a commit to EreMaijala/vufind that referenced this pull request Jun 17, 2015
@EreMaijala
Merge pull request vufind-org#3
ed920b4
demiankatz added a commit that referenced this pull request Jun 25, 2015
@demiankatz
try #3
1a370e8
demiankatz pushed a commit that referenced this pull request Aug 4, 2015
@ltog @demiankatz
Enable less parser caching (#3)
3e23245 
The code already contained everything to enable less parser caching but the function Regen() was called instead of Get() for no apparent reason. Regen() is basically a wrapper around Get() that disables caching, see https://github.com/oyejorge/less.php/blob/master/lib/Less/Cache.php .
demiankatz added a commit to olli-gold/vufind that referenced this pull request Aug 19, 2015
@demiankatz
Merge pull request vufind-org#3 from qlumbus/patch-5
515d592 
getItemCallnumber - empty check for label
demiankatz pushed a commit that referenced this pull request Oct 14, 2015
@blackmolly
Merge pull request #3 from vufind-org/ebsco-fixes-2
80a32fa 
Another update to the latest master code
@jochen-lienhard jochen-lienhard mentioned this pull request Feb 17, 2016
Closed
Piligrim2911 added a commit to malekula/vufind4LIBFL that referenced this pull request Oct 12, 2017
@Piligrim2911
Восстановление после переключения на ветку develop vufind-org#3-4
97db907
@demiankatz demiankatz mentioned this pull request May 4, 2022
Merged
13 tasks
skellamp added a commit to skellamp/vufind that referenced this pull request Feb 11, 2023
@skellamp
Error fixes in entity classes. (vufind-org#3)
6a80481
@sturkel89 sturkel89 mentioned this pull request Dec 18, 2023
Merged
3 tasks
dltj added a commit to dltj/vufind that referenced this pull request Oct 7, 2024
@dltj
Handle empty cat_password columns in user row
339e19e 
This corrects a problem with new VuFind users being created from a Shibboleth login. Without this change, we see this exception and stack trace:

```
2024-10-07T14:04:23-04:00 ERR (3): VuFind\Auth\Manager: Laminas\Db\RowGateway\Exception\InvalidArgumentException: Not a valid column in this row: cat_password in /usr/local/vufind-pmalibrary/vendor/laminas/laminas-db/src/RowGateway/AbstractRowGateway.php:294
Stack trace:
#0 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Db/Row/User.php(964): Laminas\Db\RowGateway\AbstractRowGateway->__get()
vufind-org#1 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Auth/ILSAuthenticator.php(224): VuFind\Db\Row\User->getRawCatPassword()
vufind-org#2 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Auth/Shibboleth.php(240): VuFind\Auth\ILSAuthenticator->getCatPasswordForUser()
vufind-org#3 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Auth/Manager.php(743): VuFind\Auth\Shibboleth->authenticate()
vufind-org#4 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Controller/MyResearchController.php(217): VuFind\Auth\Manager->login()
vufind-org#5 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-mvc/src/Controller/AbstractActionController.php(72): VuFind\Controller\MyResearchController->homeAction()
vufind-org#6 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Controller/Feature/CatchIlsExceptionsTrait.php(76): Laminas\Mvc\Controller\AbstractActionController->onDispatch()
vufind-org#7 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-eventmanager/src/EventManager.php(319): VuFind\Controller\MyResearchController->onDispatch()
vufind-org#8 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-eventmanager/src/EventManager.php(177): Laminas\EventManager\EventManager->triggerListeners()
vufind-org#9 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-mvc/src/Controller/AbstractController.php(105): Laminas\EventManager\EventManager->triggerEventUntil()
vufind-org#10 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-mvc/src/DispatchListener.php(117): Laminas\Mvc\Controller\AbstractController->dispatch()
vufind-org#11 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-eventmanager/src/EventManager.php(319): Laminas\Mvc\DispatchListener->onDispatch()
vufind-org#12 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-eventmanager/src/EventManager.php(177): Laminas\EventManager\EventManager->triggerListeners()
vufind-org#13 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-mvc/src/Application.php(319): Laminas\EventManager\EventManager->triggerEventUntil()
vufind-org#14 /usr/local/vufind-pmalibrary/public/index.php(71): Laminas\Mvc\Application->run()
```
demiankatz pushed a commit to dltj/vufind that referenced this pull request Oct 8, 2024
@dltj @demiankatz
Handle empty cat_password columns in user row
b7f36aa 
This corrects a problem with new VuFind users being created from a Shibboleth login. Without this change, we see this exception and stack trace:

```
2024-10-07T14:04:23-04:00 ERR (3): VuFind\Auth\Manager: Laminas\Db\RowGateway\Exception\InvalidArgumentException: Not a valid column in this row: cat_password in /usr/local/vufind-pmalibrary/vendor/laminas/laminas-db/src/RowGateway/AbstractRowGateway.php:294
Stack trace:
#0 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Db/Row/User.php(964): Laminas\Db\RowGateway\AbstractRowGateway->__get()
vufind-org#1 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Auth/ILSAuthenticator.php(224): VuFind\Db\Row\User->getRawCatPassword()
vufind-org#2 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Auth/Shibboleth.php(240): VuFind\Auth\ILSAuthenticator->getCatPasswordForUser()
vufind-org#3 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Auth/Manager.php(743): VuFind\Auth\Shibboleth->authenticate()
vufind-org#4 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Controller/MyResearchController.php(217): VuFind\Auth\Manager->login()
vufind-org#5 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-mvc/src/Controller/AbstractActionController.php(72): VuFind\Controller\MyResearchController->homeAction()
vufind-org#6 /usr/local/vufind-pmalibrary/module/VuFind/src/VuFind/Controller/Feature/CatchIlsExceptionsTrait.php(76): Laminas\Mvc\Controller\AbstractActionController->onDispatch()
vufind-org#7 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-eventmanager/src/EventManager.php(319): VuFind\Controller\MyResearchController->onDispatch()
vufind-org#8 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-eventmanager/src/EventManager.php(177): Laminas\EventManager\EventManager->triggerListeners()
vufind-org#9 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-mvc/src/Controller/AbstractController.php(105): Laminas\EventManager\EventManager->triggerEventUntil()
vufind-org#10 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-mvc/src/DispatchListener.php(117): Laminas\Mvc\Controller\AbstractController->dispatch()
vufind-org#11 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-eventmanager/src/EventManager.php(319): Laminas\Mvc\DispatchListener->onDispatch()
vufind-org#12 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-eventmanager/src/EventManager.php(177): Laminas\EventManager\EventManager->triggerListeners()
vufind-org#13 /usr/local/vufind-pmalibrary/vendor/laminas/laminas-mvc/src/Application.php(319): Laminas\EventManager\EventManager->triggerEventUntil()
vufind-org#14 /usr/local/vufind-pmalibrary/public/index.php(71): Laminas\Mvc\Application->run()
```
@maccabeelevine maccabeelevine mentioned this pull request Oct 23, 2024
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@misilot @demiankatz