Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ZCS-16412: Added command to manage the proxy timeout #175

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

swathy-shaji
Copy link

@swathy-shaji swathy-shaji commented Jan 8, 2025

Ticket - ZCS-16412

To make the proxy time-out configurable
We are providing the privilege to the user to manage this value through zmlicensectl --service command

To get the proxy timeout - zmlicensectl --service getProxyTimeout
To set the proxy timeout - zmlicensectl --service setProxyTimeout=60

Please note : We are setting the value to 3 properties internally, timeout, socketTimeout, readTimeout

Output

zimbra@telus-dev-1:~$ zmlicensectl --service setProxyTimeout=20
Error: Proxy timeout must be greater than 60.
zimbra@telus-dev-1:~$ zmlicensectl --service setProxyTimeout=
Error: Proxy timeout value is missing.
zimbra@telus-dev-1:~$ zmlicensectl --service setProxyTimeout=70
zimbra@telus-dev-1:~$ zmlicensectl --service getProxyTimeout
Current proxy timeout: 70 seconds
zimbra@telus-dev-1:~$
zimbra@telus-dev-1:~$ zmlicensectl --service help
Usage: /opt/zimbra/bin/zmlicensectl [option...]

   --service start|restart|stop|status
   --service setLogLevel=INFO|DEBUG|ERROR|WARN
   --service setOfflineMode=true|false
   --service setProxyTimeout=70 (any numeric value greater than 60 is acceptable)
   --service getLogLevel
   --service getOfflineMode
   --service getProxyTimeout
   --nalpeiron start|restart|stop|status
   --exportOfflineLicenseData | -exportOfflineLicenseData --exportStartTime YYYY/MM/DD --exportEndTime YYYY/MM/DD (offline only feature)
   --clearLicenseWorkDir
   --checkNalpeironConnectionStatus
zimbra@telus-dev-1:~$

@CLAassistant
Copy link

CLAassistant commented Jan 8, 2025

CLA assistant check
All committers have signed the CLA.

@swathy-shaji swathy-shaji force-pushed the bugfix/ZCS-16412 branch 2 times, most recently from 742346e to 4ddf33b Compare January 8, 2025 14:44
@@ -64,10 +64,15 @@ startLicenseDaemonService()
if [ "x$zimbra_license_daemon_offline_mode" = "x" ]; then
zimbra_license_daemon_offline_mode=false
fi
if [ "x$zimbra_license_proxy_timeout" = "x" ]; then

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants