RFR: 1660: Add confname field to slack log config

Erik Joelsson erikj at openjdk.org
Fri Nov 4 13:54:37 UTC 2022


On Fri, 4 Nov 2022 13:31:31 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hey Erik, nice work! Just two questions on the patch:
> 
> * why keep the field and config value `username`? If modern Slack hooks do not allow a username to be chosen, then shouldn't we remove that property from the config and the corresponding field from `BotSlackHandler`?

I'm not sure if this is a property of all slack webhooks or just the one app we are able to use at the moment. I figured we could leave the code in there for now and just not configure a username. 

> * I would probably have used the name `prefix` instead of `configname` for the configuration setting, it is an arbitrary prefix that we just happen to set to the name of the configuration 😄

That is a good idea. I will change it.

-------------

PR: https://git.openjdk.org/skara/pull/1413


More information about the skara-dev mailing list