Skip to content

Conversation

alexandreferris
Copy link
Member

  • Changed Register Client type from permanent to temporary

PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Issues

We were registering the App client as permanent which means there is a limit of 7 existing clients, and if so, it was not being handled

Solutions

For now we decided to just change the Register Client type from permanent to temporary.

"temporary" means:

temporary. A temporary client is created when you log in via the web application and choose not to be remembered. A user can have only one temporary client so any previous temporary client will be deleted when a new one is created.

We will be using this type for now and monitor it closely.

* Changed Register Client type from `permanent` to `temporary`
@alexandreferris alexandreferris requested a review from a team as a code owner August 8, 2025 13:45
@alexandreferris alexandreferris self-assigned this Aug 8, 2025
@alexandreferris alexandreferris merged commit e1e3b79 into main Aug 8, 2025
2 checks passed
@alexandreferris alexandreferris deleted the chore/handle_too_many_clients branch August 8, 2025 13:52
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