Bug 45975 - setup_saml_sp.py produces too much error output
setup_saml_sp.py produces too much error output
Status: NEW
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 5.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: UMC maintainers
UMC maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2018-01-03 18:20 CET by Arvid Requate
Modified: 2020-06-22 11:09 CEST (History)
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):
Max CVSS v3 score:


Attachments
autotest-091-master-s4.log (171.25 KB, text/x-log)
2018-01-03 18:20 CET, Arvid Requate
Details
redirect_output.patch (1.22 KB, patch)
2018-01-03 18:22 CET, Arvid Requate
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arvid Requate univentionstaff 2018-01-03 18:20:47 CET
Created attachment 9336 [details]
autotest-091-master-s4.log

In case the SAML IDP Metadata download fails for some reason, the setup_saml_sp.py produces too much error output, see attached extract from

http://jenkins.knut.univention.de:8080/job/UCS-4.3/job/UCS-4.3-0/job/AutotestJoin/SambaVersion=s4,Systemrolle=master/19/artifact/test/autotest-091-master-s4.log


In this case I guess this issue is amplified by the way ucs-ec2-tools/ucs-ec2-create handles the output, but setup_saml_sp.py should probably not output the error message of all 60 attempts but just the final one (and possibly the first).
Comment 1 Arvid Requate univentionstaff 2018-01-03 18:22:00 CET
Created attachment 9337 [details]
redirect_output.patch

Something like this could be done.