nextcloud/passman

Sans serif font makes passwords confusable

Open

#562 opened on Mar 3, 2019

View on GitHubContributor guide
 (1 comment) (0 reactions) (0 assignees)JavaScript (115 forks)auto 404
enhancementgood first issuevisual

Repository metrics

Stars
 (821 stars)
PR merge metrics
 (PR metrics pending)

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

Research direction
Locate the CSS stylesheet for password input fields and change the font family to a monospace font to make characters like 'I' and 'l' distinguishable.
Tech stack
javascriptcss
Domain
frontend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
CSSHTML
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
90