Username case

[clehner]

Re: https://matrix.to/#/!vCpZvJFNjuiBkDAvEs:cadair.com/$BBQ59q2vYuxYbJw794pbRnJRREfNmQEnuXQGKSzTBI0, https://matrix.to/#/!vCpZvJFNjuiBkDAvEs:cadair.com/$9tAWMGLZh_ooL1xdIE66B4ICey_T4BYa74MFG5XamzU

The bridge registers usernames containing uppercase characters.
I think this may have been causing a problem with Conduit (also reported here: https://gitlab.com/famedly/conduit/-/issues/295).

For now I made the following local change:

diff --git a/src/SlackGhostStore.ts b/src/SlackGhostStore.ts
index 91ed109..f5dd95f 100644
--- a/src/SlackGhostStore.ts
+++ b/src/SlackGhostStore.ts
@@ -55,7 +55,7 @@ export class SlackGhostStore {
     }

     public getUserId(id: string, teamDomain: string): string {
-        const localpart = `${this.config.username_prefix}${teamDomain.toLowerCase()}_${id.toUpperCase()}`;
+        const localpart = `${this.config.username_prefix}${teamDomain.toLowerCase()}_${id.toLowerCase()}`;
         return `@${localpart}:${this.config.homeserver.server_name}`;
     }

[jaller94]

User IDs are limited to a few specific characters. Upper case characters are not allowed.
https://spec.matrix.org/v1.3/appendices/#user-identifiers

[Half-Shot]

I think the state of play here will be to create new users with lowercase IDs (and ensure the bridge is case insensitive). Existing Matrix ghost users will remain uppercase for now (which presumably won't be an issue, as they already exist).

[clehner]

Related issue on mx-puppet-slack: https://gitlab.com/mx-puppet/slack/mx-puppet-slack/-/issues/108