DanielLeens commented on issue #12093:
URL: https://github.com/apache/seatunnel/issues/12093#issuecomment-5551704290

   Thanks @Nikk8091. I confirmed this against current dev: `findConversation()` 
reads `OAUTH_TOKEN`, while `publishMessage()` still supplies `SLACK_CHANNEL` to 
`chatPostMessage()`. There is no open PR for this issue.
   
   Please open a focused fix that changes only the publish authentication 
argument and adds a deterministic unit test around the generated request. The 
test should prove that the OAuth token, resolved channel ID, and message text 
are mapped to their own SDK fields without using a live workspace or exposing 
token values in diagnostics. Keep the existing option names and channel lookup 
behavior unchanged, then link the PR here.


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