wyona / yanel

http://www.yanel.org
Apache License 2.0
10 stars 5 forks source link

Session IDs should be hashed instead displaying actual value #30

Closed michaelwechner closed 12 years ago

michaelwechner commented 12 years ago

One can have a look at the current sessions, e.g.

http://127.0.0.1:8080/yanel/from-scratch-realm/yanel/session-manager.html

but for security reasons we don't want to display the actual session IDs, hence we should hash the IDs inside

src/resources/sessionmanager/src/java/org/wyona/yanel/impl/resources/sessionmanager/SessionManagerResource.java

csstaub commented 12 years ago

I don't see why we should display any IDs in the first place (plain or hashed). What's the point of having them in the interface?

michaelwechner commented 12 years ago

Good point, but we quite probably need it to generate notifications between administrator and individual user.

csstaub commented 12 years ago

Implemented some simple hashing

csstaub commented 12 years ago

Requires commons-codec 1.4 to be present, pushed into master branch

michaelwechner commented 12 years ago

Thanks very much.