LauraXia123 opened a new pull request, #12621:
URL: https://github.com/apache/gravitino/pull/12621
### What changes were proposed in this pull request?
- Added three optional server-side UI session timeout configurations:
- gravitino.server.webserver.sessionIdleTimeoutMs
- gravitino.server.webserver.sessionMaxDurationMs
- gravitino.server.webserver.sessionIdleWarningLeadMs
- Added the configurations to conf/gravitino.conf.template with commented
defaults and descriptions.
- Updated the UI to resolve values in this order:
1. /configs API
2. Corresponding NEXT_PUBLIC_* environment variable
3. Built-in default value
- Removed gravitino.authorization.serviceAdmins from visibleConfigs; it is
already exposed automatically by
the /configs API when authorization is enabled.
### Why are the changes needed?
Session timeout behavior previously depended only on UI environment
variables and hardcoded defaults. This
made deployment-specific configuration inconvenient and required
rebuilding or separately configuring the UI.
The server configuration now provides a centralized way to configure idle
timeout, maximum session duration,
and warning lead time.
Fix: #12588
### Does this PR introduce any user-facing change?
Yes.
- Adds three optional server configuration keys for UI session management.
- Existing UI environment variables and defaults remain supported as
fallbacks.
- gravitino.authorization.serviceAdmins remains available through /configs
when authorization is enabled; it
no longer needs to be listed in gravitino.server.visibleConfigs.
- No public REST API shape or authentication API changes are introduced.
### How was this patch tested?
From web-v2/web:
pnpm test
pnpm lint
--
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]
