Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A_25538: Too many Invites #295

Open
nikzen opened this issue Jan 29, 2025 · 1 comment
Open

A_25538: Too many Invites #295

nikzen opened this issue Jan 29, 2025 · 1 comment

Comments

@nikzen
Copy link

nikzen commented Jan 29, 2025

Hallo,
wir haben einen kleinen Verbesserungsvorschlag für diese AFO.

Anforderung

Ist das Attribute "invite" mit mehr als einem Element befüllt, dann MUSS der Messenger-Proxy an den TI-M Client das folgenden JSON-Objekt zurückgeben:

Responsecode 400

 { 
 "errcode": "M_FORBIDDEN", 
 "error": "Beim Anlegen eines Raumes darf maximal ein Teilnehmer direkt eingeladen werden"
 }

Problem

Aus unserer Sicht hat diese Anforderung 2 Probleme:

  1. M_FORBIDDEN hat den Responsecode 403 und nicht 400.
  2. Die Fehlernachricht sollte in englischer Sprache zurückgegeben werden. Localization in deutsch sollten optional sein.
@Johennes
Copy link
Contributor

  1. M_FORBIDDEN hat den Responsecode 403 und nicht 400.

Guter Punkt. Alternativ fände ich 400 (Bad Request) mit einem anderen Code auch nicht falsch.

Die Fehlernachricht sollte in englischer Sprache zurückgegeben werden. Localization in deutsch sollten optional sein.

Vielleicht könnten wir den Inhalt von error sogar komplett unspezifiziert lassen? Wenn es einen Grund gibt diesen Fehlertyp programmatisch zu erkennen, sollten wir einen dedizierten errcode verwenden. Zur Anzeige taugt der error wegen der von dir angesprochenen Localization sowieso höchstens als Fallback. Für lokalisierbare Fehler gibt es MSC4176 und dort ist die Sprache von error ebenfalls explizit unbestimmt.

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

No branches or pull requests

2 participants