Skip to content

fix(slack): remove redundant token parameter causing manual auth prompts - #124

Merged
michaelelrom merged 1 commit into
itential:mainfrom
jerry-itential:fix/slack-remove-redundant-token-param
Sep 17, 2026
Merged

michaelelrom merged 1 commit into
itential:mainfrom
jerry-itential:fix/slack-remove-redundant-token-param

Conversation

@jerry-itential

Copy link
Copy Markdown
Contributor

Summary

  • Removes the token parameter (header or query, depending on the operation) from all 60 operations that had one
  • Auth is already fully handled by the botToken security scheme (http/bearer) declared globally — this parameter was a leftover duplicate from the vendor's original per-method docs
  • Confirmed end-to-end: with this parameter present, Itential's integration config surfaces it as a required input on every task instead of using the configured bot token credential; removing it lets the configured credential handle auth as expected

Test plan

  • Imported the spec into an Itential Platform integration instance
  • Configured botToken with a real Slack bot token
  • Called chat.postMessage — succeeded without any per-task token input, using only the integration's configured credential

Every operation already authenticates via the botToken security scheme
(components.securitySchemes.botToken, http/bearer). The per-operation
`token` parameter (header on write operations, query on several read
operations) duplicated that, causing platform to expect a token input
on every task instead of relying on the configured integration credential.
@michaelelrom
michaelelrom merged commit a54717f into itential:main Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants