MrtnBalazs opened a new pull request, #615:
URL: https://github.com/apache/knox/pull/615

   ## What changes were proposed in this pull request?
   
   `ConcurrentSessionVerifier` is changed to be used and initilaized like a 
service, because initializing it in `GatewayServer` `main()` felt out of place. 
It is also modified to store the tokens issued at login so the expiry can be 
tracked since we don t have an event to subscribe to when the tokens expire. 
The verifier was put in `WebSSOResource` and `WebSSOutResource` to count the 
issued tokens when logged in or out.
   
   ## How was this patch tested?
   
   I tested it manually by configuring `gateway-site.xml` and logging in and 
out from different browsers.
   Configuration:
   ```
   <property>
           <name>gateway.non.privileged.users</name>
           <value>tom,guest</value>
    </property>
    <property>
           <name>gateway.privileged.users</name>
           <value>admin</value>
     </property>
     <property>
           <name>gateway.privileged.users.concurrent.session.limit</name>
           <value>2</value>
     </property>
     <property>
           <name>gateway.non.privileged.users.concurrent.session.limit</name>
           <value>1</value>
     </property>
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to