Opened 2 years ago

Closed 21 months ago

Last modified 21 months ago

#1487953 closed Bugs (fixed)

security lack with http_authtentication

Reported by: Lars Owned by:
Priority: 5 Milestone: 0.6-beta
Component: Security Version: 0.5.3
Severity: normal Keywords: security
Cc: singularita@…

Description

Hi,

when the http_authentication plugin is enabled and you log out you can just reenter the roundcube url and are automatically logged in as the previous user.

I'm not an http expert, but there should the http authentication be expired/killed or this behaviour should be mentioned some where.

sessions:
log in with http_auth.. enabled as mustermann
log out
log in as someone else (musterfrau) and you're logged in as mustermann (because aof the plugin)

Thanks
Lars

Change History (5)

comment:1 Changed 23 months ago by thomasb

  • Priority changed from 1 - Highest to 5
  • Severity changed from critical to normal

It's the nature of browsers that HTTP authentication headers are automatically sent in subsequent requests until the browser is closed. Please note that this is an experimental plugin which only re-uses an existing http auth but does not create it. Thus, wherever this authentication was created, this is the place to reset it again. Our plugin can therefore just define a redirect URL where the user is sent to after logging out of Roundcube.

comment:2 Changed 23 months ago by balert

you could just request sth like http://reset:reset@<roundcoube-url>?Logout=1 when logout is triggered, to just reset the currently used username and password to both "reset". the browser then would "forget" or better overwrite the actual user credentials.

comment:3 Changed 22 months ago by bilbo

  • Cc singularita@… added

It seems to add myself to CC, I have to write a valid comment. Please consider fixing the trac, so that if someone only wants to add himself to cc: for the bug, it is not necessary to actually write any comment.

Sometimes this works, but sometimes I get message like this:

Submission rejected as potential spam (Akismet says content is spam, BlogSpam? says content is spam (Too few words))

comment:4 Changed 21 months ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r5093/svn. PHP_AUTH_* data isn't used when user is provided.

comment:5 Changed 21 months ago by thomasb

Added logout redirect option in r5121/svn

The page redirected to can then reset HTTP auth if desired using hacks as suggested here: http://stackoverflow.com/questions/31326/is-there-a-browser-equivalent-to-ies-clearauthenticationcache

Note: See TracTickets for help on using tickets.