!NPRUEisLjcaMtHIzDr:kamax.io

mxisd

259 Members
Federated Matrix Identity Server | https://github.com/kamax-io/mxisd | Version: 0.8.2 | Dev discussions: #mxisd-dev:kamax.io | Related: #matrix-identity:matrix.org104 Servers

Load older messages


SenderMessageTime
11 Jun 2019
@max:kamax.ioMaximus alright, so I'll stick with 403 + M_FORBIDDEN which is the standard way to say "yes you provided good credentials but no, you're not allowed" 18:08:05
@max:kamax.ioMaximusHopefully synapse has been made to handle those things in a standardized way also18:08:32
@travis:t2l.ioTravisRsure, it'll pass it straight through to the user as per the spec.18:08:59
@max:kamax.ioMaximusI mean until v0.99.5.2, users just got "unknown server error"18:09:06
@max:kamax.ioMaximusand I haven't seen a commit that could solve that for v1.0.018:09:29
@max:kamax.ioMaximusand latest Riot doens't allow to use another other method of unbind except for HS auth, so nothing changed it seems?18:10:31
@travis:t2l.ioTravisRcorrect, Riot uses HS auth because it is deemed safe enough for the time being. If mxisd doesn't want to trust the homeserver, that's it's choice.18:12:38
@max:kamax.ioMaximusso nothing changed and mxisd is compliant, despite you claiming it is not18:13:45
@travis:t2l.ioTravisRMy last information said it was not compliant. If it is compliant, great! If it isn't, that's your problem.18:14:16
@olmari:hacklab.fi@olmari:hacklab.fiIf specs allow this choice and riot does not, then indeed riot would be to blame.. for this specific issue.. now how to make things not fail because of this would be cool to resolve (assuming that is issue in itself)18:16:47
@olmari:hacklab.fi@olmari:hacklab.fiAs a bystander thoughts, in case anyone cares :)18:17:27
@max:kamax.ioMaximus TravisR: still hoping to know which HTTP status code should be used if both methods are supported, but one is not authorized for that specific request 18:23:27
@travis:t2l.ioTravisR403, as you said.18:23:40
@travis:t2l.ioTravisRjust like regular HTTP18:23:48
@max:kamax.ioMaximusAny reason that's not in the spec then?18:24:22
@travis:t2l.ioTravisRnot wanting to explain how HTTP status codes are inherited for every single endpoint ever18:25:10
@max:kamax.ioMaximusso any HTTP status code is valid for any endpoint in the spec?18:25:37
@travis:t2l.ioTravisRwithin reason, obviously.18:25:52
@max:kamax.ioMaximusSo how can mxisd not be compliant?18:26:10
@max:kamax.ioMaximuswhat am I supposed to change?18:26:19
@travis:t2l.ioTravisRI already said that my last source of information said it was non-compliant18:26:33
@max:kamax.ioMaximus yes but non-compliant how 18:26:42
@max:kamax.ioMaximuswhat am I missing18:26:51
@travis:t2l.ioTravisRThe returned error was incompatible with the spec at the time.18:27:10
@max:kamax.ioMaximusHow so? This is the first time the endpoint is in a spec release and you just confirmed any status code can be used18:27:44
@travis:t2l.ioTravisRI don't see how this line of questioning matters18:28:06
@max:kamax.ioMaximusI would like to make mxisd compliant18:28:23
@max:kamax.ioMaximusI'm not seeing how it's non-compliant18:28:29
@travis:t2l.ioTravisRIf you're compliant now: congratulations. I won't be performing an audit to make that decision for you.18:28:32
@max:kamax.ioMaximusClearly some audit was done at some point, but you're not saying what that said except "non-compliant"18:30:13

Show newer messages


Back to Room ListRoom Version: