RedHatInsights / yggdrasil

GNU General Public License v3.0
21 stars 37 forks source link

Allow hyphenated names, with a warning #216

Closed subpop closed 6 months ago

subpop commented 7 months ago

The D-Bus name specification does not allow hyphens (-) in names or paths. Workers should really switch to using compliant names. However, in order to maintain compatibility with existing services, when a message is received for a worker, the name is passed through a scrubbing function that cleans up the name. As of this PR, this scrub function replaces hyphens with underscores and returns an error. This error is then logged. These log messages should hopefully encourage service authors to switch to using compatible names.

Closes: #210