nextcloud / nextcloud/passman

Export should escape quote characters used in CSV

Open
#419 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

bug Import/Export related needs testing
Dominant language
JavaScript
Stars
823
Forks
117
Avg merge
1d 21h
Merged PRs (30d)
8

Description

Bug report

Steps to reproduce
  1. Add passwords with ".
  2. Export as CSV.
Expected behaviour

Special characters such as ", which may clash with the csv format should be exported.

Actual behaviour

It does not do so…

Configuration

Passman version: 2.1.4

Web server: nginx

Database: MySQL

PHP version: 7.0

cloud server: Nextcloud

cloud version: 12


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue does not name a file or test. Reproduce the CSV export with a password containing a quote, then locate the export entry point and inspect the generated field quoting. Done means exported CSV preserves quote characters and can be parsed correctly by a CSV reader.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, mysql, php
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.