Skip to content

Commit

Permalink
Simplifying Web UI session timeout configuration
Browse files Browse the repository at this point in the history
The web.xml files for PKI webapps have been modified to remove
hard-coded <session-timeout> parameters. The webapps will now
use the timeout defined in /etc/pki/<instance>/web.xml.

https://pagure.io/dogtagpki/issue/3084
  • Loading branch information
edewata committed Dec 13, 2018
1 parent cc89bf5 commit 7cb7e10
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 55 deletions.
10 changes: 0 additions & 10 deletions base/ca/shared/webapps/ca/WEB-INF/web.xml
Expand Up @@ -2622,16 +2622,6 @@
<url-pattern> /ee/ca/pkiclient </url-pattern>
</servlet-mapping>

<!-- ==================== Default Session Configuration =============== -->
<!-- You can set the default session timeout (in minutes) for all newly -->
<!-- created sessions by modifying the value below. -->
<!-- -->
<!-- To disable session timeouts for this instance, set a value of -1. -->

<session-config>
<session-timeout>30</session-timeout>
</session-config>

<security-constraint>
<web-resource-collection>
<web-resource-name>Account Services</web-resource-name>
Expand Down
10 changes: 0 additions & 10 deletions base/kra/shared/webapps/kra/WEB-INF/web.xml
Expand Up @@ -1059,16 +1059,6 @@
</servlet-mapping>
[PKI_CLOSE_STANDALONE_COMMENT]

<!-- ==================== Default Session Configuration =============== -->
<!-- You can set the default session timeout (in minutes) for all newly -->
<!-- created sessions by modifying the value below. -->
<!-- -->
<!-- To disable session timeouts for this instance, set a value of -1. -->

<session-config>
<session-timeout>30</session-timeout>
</session-config>

<security-constraint>
<web-resource-collection>
<web-resource-name>Account Services</web-resource-name>
Expand Down
11 changes: 0 additions & 11 deletions base/ocsp/shared/webapps/ocsp/WEB-INF/web.xml
Expand Up @@ -693,17 +693,6 @@
</servlet-mapping>
[PKI_CLOSE_STANDALONE_COMMENT]


<!-- ==================== Default Session Configuration =============== -->
<!-- You can set the default session timeout (in minutes) for all newly -->
<!-- created sessions by modifying the value below. -->
<!-- -->
<!-- To disable session timeouts for this instance, set a value of -1. -->

<session-config>
<session-timeout>30</session-timeout>
</session-config>

<security-constraint>
<web-resource-collection>
<web-resource-name>Account Services</web-resource-name>
Expand Down
4 changes: 0 additions & 4 deletions base/server/share/webapps/pki/WEB-INF/web.xml
Expand Up @@ -45,8 +45,4 @@
</user-data-constraint>
</security-constraint>

<session-config>
<session-timeout>30</session-timeout>
</session-config>

</web-app>
10 changes: 0 additions & 10 deletions base/tks/shared/webapps/tks/WEB-INF/web.xml
Expand Up @@ -372,16 +372,6 @@
<url-pattern> /admin/tks/getStatus </url-pattern>
</servlet-mapping>

<!-- ==================== Default Session Configuration =============== -->
<!-- You can set the default session timeout (in minutes) for all newly -->
<!-- created sessions by modifying the value below. -->
<!-- -->
<!-- To disable session timeouts for this instance, set a value of -1. -->

<session-config>
<session-timeout>30</session-timeout>
</session-config>

<security-constraint>
<web-resource-collection>
<web-resource-name>Account Services</web-resource-name>
Expand Down
10 changes: 0 additions & 10 deletions base/tps/shared/webapps/tps/WEB-INF/web.xml
Expand Up @@ -137,16 +137,6 @@
<url-pattern>/tps</url-pattern>
</servlet-mapping>

<!-- ==================== Default Session Configuration =============== -->
<!-- You can set the default session timeout (in minutes) for all newly -->
<!-- created sessions by modifying the value below. -->
<!-- -->
<!-- To disable session timeouts for this instance, set a value of -1. -->

<session-config>
<session-timeout>30</session-timeout>
</session-config>

<security-constraint>
<web-resource-collection>
<web-resource-name>Account Services</web-resource-name>
Expand Down

0 comments on commit 7cb7e10

Please sign in to comment.