(WIP) Remove undocumented ST2_AUTH_TOKEN from st2chatops#181
(WIP) Remove undocumented ST2_AUTH_TOKEN from st2chatops#181
Conversation
`ST2_AUTH_TOKEN` is not documented in `st2chatops.env` and because the token either has a long expiry time or the expiry time is so short the ChatOps service is restarted all the time to pick up the config changes. It is no point to keep it, therefore remove this auth method from hubot-stackstorm entirely.
|
I don't see any significant benefit in removing token comparing to other tasks, but not opposted with the change itself. To make it happen, please check other StackStorm repos if we use any ST2_AUTH_TOKEN with chatops, for example in tests, CI, Docker images or other possible places. Additionally, don't forget about changelog record and package version change. |
Thanks for point it out and you had good point. Already found some test codes depend on ST2_AUTH_TOKEN. |
|
I found this PR pretty minimal and easy to change, I'm probably missing some background discussion around #133. As for other dependent repos that rely on token in chatops, - it would be nice to see those PRs first. |
Good point again, thanks! |
|
|
ST2_AUTH_TOKENis not documented inst2chatops.envand because the token either has a long expiry time or the expiry time is so short the ChatOps service is restarted all the time to pick up the config changes. It is no point to keep it, therefore remove this auth method from hubot-stackstorm entirely.