Default options don't secure login?

fletchgqc - May 31, 2008 - 15:09
Project:Secure Pages
Version:6.x-1.x-dev
Component:Miscellaneous
Category:support request
Priority:normal
Assigned:Unassigned
Status:reviewed & tested by the community
Description

I just enabled the module and noticed by default there is a config line to secure the url:
user/*

On my site I log in by visiting /user - which is the sort of "default" isn't it? Anyway I was able to log in without encountering an HTTPS page. So I changed the config to user* and now I can only log in over HTTPS. Shouldn't this setting be in the defaults? Isn't that the first thing you would want to secure?

#1

VM - May 31, 2008 - 15:10

I assume you log in through user because you've disabled the login block ?

do you get an https when using the block ?

#2

fletchgqc - June 1, 2008 - 11:50

1. Yes, correct.
2. Haven't tried, it's not really relevant for me.

#3

VM - June 1, 2008 - 13:27

while it may not be relevant for you, it is relevant to the request and your question about whether or not it should be a default setting.

#4

compudaze - February 19, 2009 - 21:46

How would I secure login with this module?

#5

VM - February 19, 2009 - 21:52
Status:active» fixed

I'd think you would want to investigate the securelogin.module

#6

fletchgqc - February 20, 2009 - 09:52

Or just add /user to the pages to be viewed over HTTPS.

#7

System Message - March 6, 2009 - 10:00
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

#8

DamienMcKenna - April 27, 2009 - 12:02

The v6.x-1.7 release secures the user/* pages by default.

#9

fletchgqc - April 27, 2009 - 14:40

"user/*" does not match "user"

The ideal solution would be two lines:
user/*
user

#10

DamienMcKenna - April 27, 2009 - 15:00
Status:closed» needs review

fletchgqc: Correct. I've attached a tiny patch for the D6 branch to add it in. It also separates the main admin page vs the sub-pages, to have them listed in the same format.

AttachmentSize
securepages-n264987.patch 1.3 KB

#11

Matt V. - June 16, 2009 - 20:16
Status:needs review» reviewed & tested by the community

I applied the patch listed in comment #10 to version 6.x-1.8 of the module. It applied without any trouble and appears to work as advertised. I think it is an important patch because someone who just accepts the defaults after installing this module could easily be under the mistaken impression that user logins are secure, when they may not be.

 
 

Drupal is a registered trademark of Dries Buytaert.