* fix: correct non-standard 'yml' tag to 'yaml' in RedisOpts
* fix: apply MaxActive config to Redis PoolSize
* fix: clarify config semantics, enhance timeout & pool options, and maintain backward compatibility
* test: update unit test for redis
* refactor: apply suggestions from code review
* test: add comprehensive coverage for redis
* refactor: resolve funlen linter errors in redis unit tests
* refactor: remove empty else-if branch in NewRedis function
* fix: improve type safety in httpWithTLS for custom RoundTripper
Add type assertion check to handle cases where DefaultHTTPClient.Transport
is a custom http.RoundTripper implementation (not *http.Transport).
This improves upon the fix in PR #844 which only handled nil Transport.
The previous code would still panic if users set a custom RoundTripper:
trans := baseTransport.(*http.Transport).Clone() // panic if not *http.Transport
Now safely handles three scenarios:
1. Transport is nil -> use http.DefaultTransport
2. Transport is *http.Transport -> clone it
3. Transport is custom RoundTripper -> use http.DefaultTransport
Added comprehensive test cases:
- TestHttpWithTLS_NilTransport
- TestHttpWithTLS_CustomTransport
- TestHttpWithTLS_CustomRoundTripper
Related to #803
* refactor: reduce code duplication and complexity in httpWithTLS
- Eliminate duplicate http.DefaultTransport.Clone() calls
- Reduce cyclomatic complexity by simplifying conditional logic
- Use nil check pattern instead of nested else branches
- Maintain same functionality with cleaner code structure
This addresses golangci-lint warnings for dupl and gocyclo.
* fix: add newline at end of http_test.go
Fix gofmt -s compliance issue:
- File must end with newline character
- Addresses golangci-lint gofmt error on line 81
This fixes CI check failure.
* feat: enhance WorkAccessToken to include AgentID for improved token management
- Added AgentID field to WorkAccessToken struct.
- Updated NewWorkAccessToken function to accept AgentID as a parameter.
- Modified access token cache key to incorporate AgentID, ensuring unique cache entries per agent.
This change improves the handling of access tokens in a multi-agent environment.
* refactor: enhance WorkAccessToken to improve cache key handling
- Updated the AgentID field in WorkAccessToken struct to clarify its optional nature for distinguishing applications.
- Modified the access token cache key construction to support both new and legacy formats based on the presence of AgentID.
- Added comments for better understanding of the cache key logic and its compatibility with historical versions.
This change improves the flexibility and clarity of access token management in multi-agent scenarios.
* feat(work): add JsSdk method for JavaScript SDK integration
- Introduced a new JsSdk method in the Work struct to facilitate the creation of a Js instance.
- This addition enhances the functionality of the Work module by enabling JavaScript SDK support.
This change improves the integration capabilities for developers working with the WeChat Work API.
* fix gofmt
* refactor(jsapi): simplify signature generation in GetConfig and GetAgentConfig methods
- Replaced the use of strconv to format the signature string with fmt.Sprintf for improved readability.
- Updated the signature generation logic in both GetConfig and GetAgentConfig methods to use a single formatted string.
This change enhances code clarity and maintains functionality in the signature generation process.
* fix gofmt
* fix gofmt
* fix gofmt
* fix gofmt
* refactor(js): correct initialization of Config in GetConfig method
- Removed redundant initialization of the Config variable and ensured it is instantiated correctly before use.
- This change improves code clarity and maintains the intended functionality of the GetConfig method.
* feat: enhance WorkAccessToken to include AgentID for improved token management
- Added AgentID field to WorkAccessToken struct.
- Updated NewWorkAccessToken function to accept AgentID as a parameter.
- Modified access token cache key to incorporate AgentID, ensuring unique cache entries per agent.
This change improves the handling of access tokens in a multi-agent environment.
* refactor: enhance WorkAccessToken to improve cache key handling
- Updated the AgentID field in WorkAccessToken struct to clarify its optional nature for distinguishing applications.
- Modified the access token cache key construction to support both new and legacy formats based on the presence of AgentID.
- Added comments for better understanding of the cache key logic and its compatibility with historical versions.
This change improves the flexibility and clarity of access token management in multi-agent scenarios.
* feat(work): add JsSdk method for JavaScript SDK integration
- Introduced a new JsSdk method in the Work struct to facilitate the creation of a Js instance.
- This addition enhances the functionality of the Work module by enabling JavaScript SDK support.
This change improves the integration capabilities for developers working with the WeChat Work API.
* fix gofmt
* feat: enhance WorkAccessToken to include AgentID for improved token management
- Added AgentID field to WorkAccessToken struct.
- Updated NewWorkAccessToken function to accept AgentID as a parameter.
- Modified access token cache key to incorporate AgentID, ensuring unique cache entries per agent.
This change improves the handling of access tokens in a multi-agent environment.
* refactor: enhance WorkAccessToken to improve cache key handling
- Updated the AgentID field in WorkAccessToken struct to clarify its optional nature for distinguishing applications.
- Modified the access token cache key construction to support both new and legacy formats based on the presence of AgentID.
- Added comments for better understanding of the cache key logic and its compatibility with historical versions.
This change improves the flexibility and clarity of access token management in multi-agent scenarios.
* feat: enhance WorkAccessToken with new constructor for AgentID support
- Introduced NewWorkAccessTokenWithAgentID function to maintain backward compatibility while allowing for AgentID usage.
- Updated NewWorkAccessToken to call the new constructor, ensuring seamless integration.
- Improved error handling in GetAccessTokenContext by checking for cache availability and handling potential errors during cache operations.
This change enhances the flexibility of access token management, particularly in multi-agent scenarios, while ensuring compatibility with existing implementations.
