Univention Bugzilla – Full Text Bug Listing |
Summary: | System info upload uses wrong url | ||
---|---|---|---|
Product: | UCS | Reporter: | Janis Meybohm <meybohm> |
Component: | Sysinfo | Assignee: | Florian Best <best> |
Status: | CLOSED FIXED | QA Contact: | Drees Dormann <dormann> |
Severity: | normal | ||
Priority: | P5 | CC: | best, gohmann, klaeser, walkenhorst, wiesenthal |
Version: | UCS 3.1 | ||
Target Milestone: | UCS 4.0-0-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
See Also: | https://forge.univention.org/bugzilla/show_bug.cgi?id=33092 | ||
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: | |||
Bug Depends on: | |||
Bug Blocks: | 31199 |
Description
Janis Meybohm
2013-04-25 14:17:22 CEST
This is most probably a server-side error (UCS 2.4 cannot upload either). The cgi.FieldStorage() is completely empty and no file is found. The server sends the "correct" (yet very uncomfortable) error message: ERROR: wrong file type or file to big This message is suppressed by the client which may be considered a bug. And the whole error handling is not very graceful. But in the end it is a problem of forge.univention.de This is the best I can come up with. It is just a guess: http://stackoverflow.com/questions/2781493/cgi-fieldstorage-always-empty-never-returns-posted-form-data Did we add new apache services? Do we use mod_python since February? By the way: max_size = configRegistry.get( 'umc/sysinfo/upload/size', '2000000' ) ... size > max_size will always be true because an int is compared to a string The real "bug" probably is that the "wrong" domain is used for uploading: - FALLBACK_UPLOAD_URL = 'https://forge.univention.de/cgi-bin/system-info-upload.py' + FALLBACK_UPLOAD_URL = 'https://forge.univention.org/cgi-bin/system-info-upload.py' https://forge.univention.de is then redirected to https://forge.univention.org on server side which cases the CGI to break. I'm going to add an exception here on the server side but we should change the domain in the code. Besides: No need for errata anymore! The wrong domain is also used by UCS 2.4. I suppose we have to support both domains for a while. Replaced forge.univention.de by forge.univention.org during working on sysinfo. Fix: svn r56967 YAML: 2014-12-18-univention-system-info.yaml upload is working YAML ok |