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).

CommentFileSizeAuthor
#12 simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-12.patch502 bytesbrianfisher
#8 simplesamlphp_auth-exclude-cli-mode-from-processing-2868578.patch443 bytessnufkin
#5 simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-5.patch475 byteskiseleva.t
#3 simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-3.patch494 byteskiseleva.t
#2 simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-2.patch750 byteskiseleva.t

Comments

kiseleva.t created an issue. See original summary.

kiseleva.t’s picture

kiseleva.t commented
StatusFileSize
new simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-2.patch750 bytes

This patch solved issue for me.

kiseleva.t’s picture

kiseleva.t commented
Version: 7.x-2.0-alpha2 » 7.x-2.x-dev
StatusFileSize
new simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-3.patch494 bytes

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

kiseleva.t’s picture

kiseleva.t commented
Status: Active » Needs review
kiseleva.t’s picture

kiseleva.t commented
StatusFileSize
new simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-5.patch475 bytes

Patch from comment 3 has wrong place check, fixed.

  • snufkin committed a3aa4af on 7.x-2.x authored by kiseleva.t 
    Issue #2868578 by kiseleva.t: Drush command terminated abnormally due to...
snufkin’s picture

snufkin commented
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!

snufkin’s picture

snufkin commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new simplesamlphp_auth-exclude-cli-mode-from-processing-2868578.patch443 bytes

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

aprohl5’s picture

aprohl5 commented

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?

a_thakur’s picture

a_thakur commented

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?

kiseleva.t’s picture

kiseleva.t commented

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.

brianfisher’s picture

brianfisher commented
StatusFileSize
new simplesamlphp_auth-exclude-cli-mode-from-processing-2868578-12.patch502 bytes

For those on 7.x-2.x, this patch fixes comment #10