Improve error messages from Filesystem::readRandomBytes()
ClosedPublic
Actions

Authored by epriestley on Nov 25 2013, 8:58 PM.

Tags

None

Referenced Files

	F19010338: D7650.id17274.diff
	Sat, Nov 22, 3:40 AM

	F18875073: D7650.id17281.diff
	Nov 5 2025, 4:42 PM

	F18828129: D7650.id.diff
	Oct 24 2025, 2:29 PM

	F18822117: D7650.diff
	Oct 23 2025, 4:21 AM

	F18619598: D7650.id.diff
	Sep 15 2025, 2:52 AM

	F18596963: D7650.diff
	Sep 13 2025, 1:53 AM

	F18590372: D7650.id17281.diff
	Sep 12 2025, 5:54 AM

	F18577977: D7650.id17274.diff
	Sep 10 2025, 8:24 PM

View All Files

Subscribers

aran

Details

Reviewers

hach-que
btrahan

Maniphest Tasks

T4153: Filesystem::readRandomBytes does not work on Windows without php_openssl.dll

Commits

rPHU5fab43a1cccd: Improve error messages from Filesystem::readRandomBytes()

Summary

Fixes T4153. Primarily, raise a better error message on Windows. Also make the function a little stricter and a little better tested.

Test Plan

Unit tests.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

Tested on Windows and verified the message appears when OpenSSL is not available.

Closed by commit rPHU5fab43a1cccd (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

filesystem/

Filesystem.php

64 lines

__tests__/

FilesystemTestCase.php

13 lines

Diff 17281

View Options

src/filesystem/Filesystem.php

View Options

src/filesystem/tests/FilesystemTestCase.php

Improve error messages from Filesystem::readRandomBytes()ClosedPublicActions