[
https://issues.apache.org/jira/browse/KNOX-3119?focusedWorklogId=965089&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-965089
]
ASF GitHub Bot logged work on KNOX-3119:
----------------------------------------
Author: ASF GitHub Bot
Created on: 07/Apr/25 15:54
Start Date: 07/Apr/25 15:54
Worklog Time Spent: 10m
Work Description: lmccay commented on code in PR #1016:
URL: https://github.com/apache/knox/pull/1016#discussion_r2031549654
##########
gateway-service-knoxtoken/src/main/java/org/apache/knox/gateway/service/knoxtoken/TokenResource.java:
##########
@@ -219,8 +220,13 @@ public int toInt() {
}
}
+ protected ServletContext wrapContextForDefaultParams(ServletContext context)
throws ServletException {
+ return context;
+ }
+
@PostConstruct
- public void init() throws AliasServiceException, ServiceLifecycleException,
KeyLengthException {
+ public void init() throws AliasServiceException, ServiceLifecycleException,
KeyLengthException, ServletException {
+ ServletContext context = wrapContextForDefaultParams(this.context);
Review Comment:
addressed.
Issue Time Tracking
-------------------
Worklog Id: (was: 965089)
Time Spent: 1h 20m (was: 1h 10m)
> Tune defaults for config params in CLIENTID API
> -----------------------------------------------
>
> Key: KNOX-3119
> URL: https://issues.apache.org/jira/browse/KNOX-3119
> Project: Apache Knox
> Issue Type: Improvement
> Components: Server
> Reporter: Larry McCay
> Assignee: Larry McCay
> Priority: Major
> Fix For: 2.2.0
>
> Time Spent: 1h 20m
> Remaining Estimate: 0h
>
> There are a number of params that should be set by default for the CLIENTID
> extension of the KNOXTOKEN API. This will reduce the likelihood of errors
> based on having to explicitly configure these params which are either
> required or the logical defaults.
> 1. knox.token.exp.server-managed should be enabled by default this service if
> based on passcode tokens and requires server state
> 2. knox.token.ttl - should be -1 by default to not expire - fight me! :)
> 3. knox.token.hash.key should be set by default as well. We need to check
> whether this already exists via the AliasService and if not we should
> generate it automatically or fail to deploy with appropriate error/exception.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
