Page MenuHomePhabricator

[keyserver] Add password sanitization tests
ClosedPublic

Authored by michal on Apr 18 2023, 8:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 10:04 PM
Unknown Object (File)
Sun, Dec 15, 10:04 PM
Unknown Object (File)
Sun, Dec 15, 10:04 PM
Unknown Object (File)
Sun, Dec 15, 10:03 PM
Unknown Object (File)
Sun, Dec 15, 9:51 PM
Unknown Object (File)
Thu, Dec 5, 2:30 AM
Unknown Object (File)
Thu, Dec 5, 2:30 AM
Unknown Object (File)
Thu, Dec 5, 2:30 AM
Subscribers

Details

Summary

The sanitizeInput function will be abstracted by a general function implementing a visitor pattern (which we will also use for the server_id<->client_id conversion function) in the later diffs. This diff adds some tests so we can check that the behaviour doesn't change.

Test Plan

Run yarn jest

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

keyserver/src/utils/validation-utils.js
64 ↗(On Diff #25271)

Thanks for getting rid of the *

Is it possible to type this any better?

keyserver/src/utils/validation-utils.js
64 ↗(On Diff #25271)

Ignore this comment, types are improved in D7488

This revision is now accepted and ready to land.Apr 21 2023, 1:39 AM