Drush command terminated abnormally due to an error with simplesamlphp [#2868578]

I've faced with error during execute drush command on server:

Drush command terminated abnormally due to an unrecoverable error. [error]
SimpleSAML_Error_Exception: Setting secure cookie on plain HTTP is not allowed. in SimpleSAML_Session::getSessionFromRequest() (line 287 of sites/all/libraries/simplesamlphp/lib/SimpleSAML/Session.php).

Comment	File	Size	Author
#12	simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-12.patch	502 bytes	brianfisher
#12
#8	simplesamlphp_auth-exclude-cli-mode-from-processing-2868578.patch	443 bytes	snufkin
#8
#5	simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-5.patch	475 bytes	kiseleva.t
#5
#3	simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-3.patch	494 bytes	kiseleva.t
#3
#2	simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-2.patch	750 bytes	kiseleva.t
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

11 April 2017 at 09:11

kiseleva.t created an issue. See original summary.

File	Size
simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-2.patch	750 bytes

Comment #3

kiseleva.t CreditAttribution: kiseleva.t as a volunteer and at DrupalJedi commented 11 April 2017 at 09:22

Version:

7.x-2.0-alpha2

» 7.x-2.x-dev

File	Size
simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-3.patch	494 bytes

Updated patch from 7.x-2.x-dev version and changed order of check.

Comment #4

kiseleva.t CreditAttribution: kiseleva.t as a volunteer and at DrupalJedi commented 11 April 2017 at 09:23

Status:

Active

» Needs review

Comment #5

kiseleva.t CreditAttribution: kiseleva.t as a volunteer and at DrupalJedi commented 11 April 2017 at 09:27

File	Size
simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-5.patch	475 bytes

Patch from comment 3 has wrong place check, fixed.

Comment #6

12 April 2017 at 09:45

snufkin committed a3aa4af on 7.x-2.x authored by kiseleva.t

Issue #2868578 by kiseleva.t: Drush command terminated abnormally due to...

Comment #7

snufkin CreditAttribution: snufkin as a volunteer commented 12 April 2017 at 09:46

Version:	7.x-2.x-dev	» 7.x-3.x-dev
Status:	Needs review	» Patch (to be ported)

Looks like an obvious fix to the problem, I'm surprised this hasn't come up before :) Thanks!

Comment #8

snufkin CreditAttribution: snufkin as a volunteer commented 12 April 2017 at 09:49

Status:

Patch (to be ported)

» Needs review

File	Size
simplesamlphp_auth-exclude-cli-mode-from-processing-2868578.patch	443 bytes

Porting the patch for 7.x-3.x. Should be trivial.

Comment #9

aprohl5 CreditAttribution: aprohl5 as a volunteer commented 23 April 2017 at 18:10

I am unable to reproduce this issue in the current 7.x-3.x-dev version of the module despite the patches above appearing to have been unapplied. is there a specific command that was erroring out in order to produce this issue?

Comment #10

a_thakur CreditAttribution: a_thakur as a volunteer and at Srijan | A Material+ Company commented 10 May 2017 at 18:08

I ran into a similar problem where I had to create a new user in update hook, due to hook_user_insert implementation in 2.x fails, does it make sense to add a similar condition there as well?

Comment #11

kiseleva.t CreditAttribution: kiseleva.t as a volunteer and at DrupalJedi commented 31 August 2017 at 05:41

For me issue appeared during automatic deployment on stage server, I think it could be drush updb or something like that.
Also it happened only when I enabled session.cookie.secure in config, before I didn't have such error.