Bug 45662 - [RESTful Import API] UMC module: don't rename CSV file
[RESTful Import API] UMC module: don't rename CSV file
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - CSV Import
UCS@school 4.2
Other Linux
: P5 normal (vote)
: UCS@school 4.2 (HTTP-API-MVP)
Assigned To: Florian Best
Daniel Tröder
:
Depends on: 45023
Blocks:
  Show dependency treegraph
 
Reported: 2017-11-07 14:36 CET by Daniel Tröder
Modified: 2017-12-21 12:23 CET (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
Who will be affected by this bug?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.046
Enterprise Customer affected?:
School Customer affected?: Yes
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 Daniel Tröder univentionstaff 2017-11-07 14:36:09 CET
The UMC module changes the filenames of the CSV files, so that they are later not recognizable anymore: K70JS3N7VTP13ZT, BL1CRR7L918MONE, 64XZ625A37URXGD, ...

When administrators have to debug import jobs, they have to look at /var/spool/ucs-school-import/jobs/$YEAR/$JOB-ID/ and they won't find any *.csv file there.

The UMC module should upload the CSV files with the filename that it had, when on the users computer. If necessary it can add a random string, but the filename must end in ".csv".
Comment 1 Florian Best univentionstaff 2017-11-15 12:06:57 CET
ucs-school-umc-import.yaml
462049a16550 | Bug #45662: Don't rename the original filename

ucs-school-umc-import (0.0.0-13)
462049a16550 | Bug #45662: Don't rename the original filename
Comment 2 Daniel Tröder univentionstaff 2017-11-20 14:42:02 CET
OK: solution: the original time name is preserved with a timestamp prepended
OK: code change
OK: manual test
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2017-12-21 12:23:01 CET
UCS@school 4.2 v6 has been released.

http://docs.software-univention.de/changelog-ucsschool-4.2v6-de.html

If this error occurs again, please clone this bug.