Bug 52305 – redirect to simplesamlphp API instead of creating a AuthNRequest to improve performance

Bug 52305 - redirect to simplesamlphp API instead of creating a AuthNRequest to improve performance


Summary:	redirect to simplesamlphp API instead of creating a AuthNRequest to improve p...

Status:	RESOLVED WONTFIX

Product:	UCS
Classification:	Unclassified
Component:	UMC (Generic)
Version:	UCS 4.4
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	UMC maintainers
QA Contact:	Dirk Wiesenthal

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2020-11-03 10:46 CET by Florian Best
Modified:	2022-06-29 13:53 CEST (History)
CC List:	1 user (show)

See Also:
What kind of report is it?:	Development Internal
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):	API change, Large environments, SAML, UCS Performance
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Florian Best

2020-11-03 10:46:26 CET

pysaml2 takes ~30ms to create the initial AuthNRequest because a subprocess calling xmldsig is necessary.

Instead of creating such AuthNRequest we could go hardcoded against the simplesamlphp API/URLs and only do a redirect to a identity provider initiated login.

Comment 1 Florian Best

2022-06-29 13:53:08 CEST

luckily we did not do this change as we are now supporting keycloak as well.