6.0.0-beta1
7/21/25

[#13464] Missing $ sign in HTTP Driver
Summary Missing $ sign in HTTP Driver
Queue Passwd
Queue Version 5.0.1
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester alex.lanz (at) rolmail (dot) net
Created 08/22/2014 (3986 days ago)
Due
Updated 08/22/2014 (3986 days ago)
Assigned
Resolved 08/22/2014 (3986 days ago)
Github Issue Link
Github Pull Request
Milestone 5.0.2
Patch No

History
08/22/2014 10:20:17 AM Jan Schneider Assigned to Jan Schneider
State ⇒ Resolved
Milestone ⇒ 5.0.2
 
08/22/2014 10:19:46 AM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit 4668e7ed4570bc9c4d1cbc3dccbd0ddf62dc4787
Author: Jan Schneider <jan@horde.org>
Date:   Fri Aug 22 12:16:11 2014 +0200

     Fix syntax error (Bug #13464).

  passwd/docs/CHANGES        |    1 +
  passwd/lib/Driver/Http.php |    2 +-
  passwd/package.xml         |    2 ++
  3 files changed, 4 insertions(+), 1 deletions(-)

http://github.com/horde/horde/commit/4668e7ed4570bc9c4d1cbc3dccbd0ddf62dc4787
08/22/2014 10:17:44 AM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit 5f6555631fd1508157b19365269a420c0cb5a542
Author: Jan Schneider <jan@horde.org>
Date:   Fri Aug 22 12:16:11 2014 +0200

     Fix syntax error (Bug #13464).

  passwd/docs/CHANGES        |    1 +
  passwd/lib/Driver/Http.php |    2 +-
  passwd/package.xml         |    2 ++
  3 files changed, 4 insertions(+), 1 deletions(-)

http://github.com/horde/horde/commit/5f6555631fd1508157b19365269a420c0cb5a542
08/22/2014 07:04:19 AM alex (dot) lanz (at) rolmail (dot) net Comment #1
Priority ⇒ 1. Low
Patch ⇒ No
Milestone ⇒
Queue ⇒ Passwd
Summary ⇒ Missing $ sign in HTTP Driver
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
In the file passwd/lib/Driver/Http.php there is a missing dollar sign 
almost at the bottom of the file. At the last check the body variable 
should be named "$body".

Acutal Code:

         if (!strpos(body, $this->_params['eval_results']['success'])) {
             throw new Passwd_Exception(_("Your password could not be 
changed."));
         }


Correct Code:

         if (!strpos($body, $this->_params['eval_results']['success'])) {
             throw new Passwd_Exception(_("Your password could not be 
changed."));
         }

Saved Queries