nextcloud / nextcloud/passman

Sans serif font makes passwords confusable

Open
#562 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement good first issue visual
Dominant language
JavaScript
Stars
823
Forks
117
Avg merge
1d 21h
Merged PRs (30d)
8

Description

Bug report

Steps to reproduce
  1. Create a password with "I" (capital i) and "l" (lower case L) and save
  2. View password entry
  3. Click "Show password"
Expected behaviour

Password can be read back without confusing letters

Actual behaviour

It's impossible to differentiate I from l.

(Also applies to O/0 to a lesser degree)

Configuration

Operating system: Android Pie and openSUSE Tumbleweed

Browser: Firefox 65.0.1 (mobile and desktop)

Extensions that might cause interference: None

Passman version: 2.2.1

cloud server: Nextcloud

cloud version: 15.0.4


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

Start by reproducing the issue in the password entry view on the Show password action, then inspect the frontend code and styles responsible for rendering visible passwords. Done means characters such as capital I, lowercase l, O, and 0 can be distinguished when the password is shown.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.