Bug 33705 - UMC-Server: revise UMCP auth response when authentication fails
UMC-Server: revise UMCP auth response when authentication fails
Status: RESOLVED WORKSFORME
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.2
Other Linux
: P5 normal (vote)
: UCS 3.x
Assigned To: UMC maintainers
:
: 35571 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-10 12:26 CET by Florian Best
Modified: 2015-11-24 11:05 CET (History)
3 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2013-12-10 12:26:36 CET
Currently these cases exists when a UMCP authentication can fail:

Wrong password:
{"status": "411 Length Required", "message": "The authentication has failed"}
→ Should be 401

LDAP down:
{"status": "411 Length Required", "message": "The authentication has failed"}
→ Should be 503

univention-management-console-server is down:
{"status": "401 Unauthorized", "message": "No permission -- see authorization schemes"}
→ Should be 503

univention-management-console-web-server is down:
503 Service Temporarily Unavailable
→ Is correct
Comment 1 Alexander Kläser univentionstaff 2013-12-12 11:39:58 CET
(In reply to Florian Best from comment #0)
> Currently these cases exists when a UMCP authentication can fail:
> 
> Wrong password:
> {"status": "411 Length Required", "message": "The authentication has failed"}
> → Should be 401

I guess 411 is OK.

> Status( 'BAD_REQUEST_AUTH_FAILED' , 411, ( 'The authentication has failed' ) ),
Comment 2 Florian Best univentionstaff 2014-08-12 20:09:53 CEST
*** Bug 35571 has been marked as a duplicate of this bug. ***
Comment 3 Florian Best univentionstaff 2015-05-07 20:42:11 CEST
(In reply to Alexander Kläser from comment #1)
> (In reply to Florian Best from comment #0)
> > Currently these cases exists when a UMCP authentication can fail:
> > 
> > Wrong password:
> > {"status": "411 Length Required", "message": "The authentication has failed"}
> > → Should be 401
> 
> I guess 411 is OK.
> 
> > Status( 'BAD_REQUEST_AUTH_FAILED' , 411, ( 'The authentication has failed' ) ),

No it is not. A HTTP client thinks that he did not send a Content-Length request header then.
Comment 4 Florian Best univentionstaff 2015-11-24 11:05:52 CET
In UCS 4.1 only 401 is used with some flags which indicated the failure (password expired, ...).