Mercurial > dive4elements > river
comparison gwt-client/src/main/java/org/dive4elements/river/client/server/auth/was/Response.java @ 5948:d7b9b3e3c61a
Make instantiation of saml.User easier.
Most of the parameters of the constructor can be taken from the
Assertion object, so there's no reason to pass them separately.
Also, trying to check the validity dates isn't useful for the single
sign on case. See comments in the hasExpired method.
| author | Bernhard Herzog <bh@intevation.de> |
|---|---|
| date | Wed, 08 May 2013 17:56:14 +0200 |
| parents | 0b092a1d136b |
| children | ea9eef426962 |
comparison
equal
deleted
inserted
replaced
| 5947:0b092a1d136b | 5948:d7b9b3e3c61a |
|---|---|
| 111 } | 111 } |
| 112 List<String> features = this.features.getFeatures( | 112 List<String> features = this.features.getFeatures( |
| 113 this.assertion.getRoles()); | 113 this.assertion.getRoles()); |
| 114 logger.debug("User " + this.username + " with features " + features + | 114 logger.debug("User " + this.username + " with features " + features + |
| 115 " successfully authenticated."); | 115 " successfully authenticated."); |
| 116 return new User(this.username, this.password, assertion.getNameID(), | 116 return new User(assertion, features, this.password); |
| 117 this.assertion.getRoles(), assertion, features); | |
| 118 } | 117 } |
| 119 } | 118 } |
| 120 // vim: set si et fileencoding=utf-8 ts=4 sw=4 tw=80: | 119 // vim: set si et fileencoding=utf-8 ts=4 sw=4 tw=80: |