Nikk8091 opened a new pull request, #12171:
URL: https://github.com/apache/seatunnel/pull/12171

   ### Purpose of this pull request
   
   Fixes #12093.
   
   SlackClient.publishMessage previously supplied the configured Slack channel 
name as the chat.postMessage authentication token. This change maps 
oauth_token, the resolved channel ID, and message text to their correct SDK 
request fields.
   
   ### Does this PR introduce any user-facing change?
   
   Yes. Slack sink message publishing now authenticates with the configured 
oauth_token instead of incorrectly using slack_channel as the token. Existing 
option names and channel lookup behavior are unchanged.
   
   ### How was this patch tested?
   
   - Added SlackClientTest to verify OAuth token, resolved channel ID, and text 
mapping without a live Slack workspace.
   - connector-slack tests: 2 passed, 0 failures.
   - connector-slack verify: passed.
   - Spotless formatting: passed.
   
   ### Check list
   
   * [x] No new Jar binary package is added.
   * [x] Documentation is not required because no configuration option or usage 
changes.
   * [x] No incompatible change is introduced.
   * [x] Existing connector registration, distribution, CI labels, E2E setup, 
and plugin configuration are unchanged.


-- 
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]

Reply via email to