Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#40 closed defect (fixed)

[trac] whitespaces in attachment names lead to broken attachment URLs

Reported by: philll Assignee: trev
Priority: P2 Milestone:
Module: Infrastructure Keywords:
Cc: Blocked By:
Blocking: Platform:
Ready: no Confidential: no
Tester: Verified working: yes
Review URL(s):

http://codereview.adblockplus.org/5719138018263040/

Description

How to reproduce

  1. Open an existing trac issue at https://issues.adblockplus.org
  2. Expand the "Attachments" section
  3. Click the "Attach File button"
  4. Click the "Choose File" button
  5. Select a file with a whitespace in its name for uploading
  6. Click the Add Attachment" button
  7. Click the file's name

Observed behaviour

The following note is displayed:

Error: Invalid Attachment
Attachment 'ticket:1: white%20space.2.png' does not exist.

Expected behaviour

The file should be displayed and/or offered for download

Change History (10)

comment:1 Changed 5 years ago by trev

Implemented a proper way to set up PATH_INFO variable: http://codereview.adblockplus.org/5719138018263040

comment:2 Changed 5 years ago by trev

  • Status changed from new to accepted

comment:3 Changed 5 years ago by trev

  • Owner set to trev
  • Status changed from accepted to assigned

comment:4 Changed 5 years ago by trev

  • Status changed from assigned to accepted

comment:5 Changed 5 years ago by trev

  • Status changed from accepted to assigned

comment:6 Changed 5 years ago by trev

  • Status changed from assigned to reviewing

comment:7 Changed 5 years ago by trev

  • Priority changed from Unknown to P2

comment:8 Changed 5 years ago by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:9 Changed 5 years ago by trev

  • in_progress set to 0
  • Ready unset
  • Review URL(s) modified (diff)

comment:10 Changed 5 years ago by philll

  • Verified working set
Note: See TracTickets for help on using tickets.