Paths

Table of Contentst

Differential D25229

Fix PHP 8.1 "explode(null)" exception which blocks rendering "Provide Multi-Factor Credentials" page
ClosedPublic
Actions

Authored by aklapper on May 13 2023, 13:48.

Tags

None

Referenced Files

	F3293926: D25229.1742926406.diff
	Mon, Mar 24, 18:13

	F3290405: D25229.1742864739.diff
	Mon, Mar 24, 01:05

	F3224961: D25229.1742089902.diff
	Sat, Mar 15, 01:51

	F3221059: D25229.1741813553.diff
	Tue, Mar 11, 21:05

	F3221058: D25229.1741813549.diff
	Tue, Mar 11, 21:05

	F3221057: D25229.1741813546.diff
	Tue, Mar 11, 21:05

	F3220313: D25229.1741808861.diff
	Tue, Mar 11, 19:47

	F3053254: D25229.1740988263.diff
	Sun, Mar 2, 07:51

View All 32 Files

Subscribers

valerio.bozzolan

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15397: PHP 8.1 "explode(null)" exception blocks rendering "Provide Multi-Factor Credentials" page

Commits

rP10fae71a042f: Fix PHP 8.1 "explode(null)" exception which blocks rendering "Provide Multi…

Summary

Passing null to the $string in explode(string $separator, string $string, int $limit) is deprecated since PHP 8.1.
Thus do not explode but create an empty array if the input is null.

Closes T15397

Test Plan

Applied this change and afterwards the "Provide Multi-Factor Credentials" page at /auth/finish/ showing the "App Code" field correctly rendered in the web browser.

Diff Detail

Repository

Branch

arcpatch-D25229_2

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 431
Build 431: arc lint + arc unit

Event Timeline

aklapper created this revision.May 13 2023, 13:48

Owners added a reviewer: O1: Blessed Committers.May 13 2023, 13:48

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan and 2 others. · View Herald TranscriptMay 13 2023, 13:48

aklapper requested review of this revision.May 13 2023, 13:48

Harbormaster completed remote builds in B413: Diff 737.May 13 2023, 13:48

valerio.bozzolan mentioned this in Wikimedia Hackaton 2023 Athens Code Sprint.May 18 2023, 10:16

Thanks - small tip here

src/applications/auth/storage/PhabricatorAuthChallenge.php
59	Also here probably the minimal change is just the above one

Thanks again for this patch

I talked with the kind user aklapper about the proposed minimal change and, hoping to be useful, I will amend that small thing

I tested this locally and this works

This revision is now accepted and ready to land.May 20 2023, 20:27

amend a small thing we discussed in live

Harbormaster completed remote builds in B431: Diff 771.May 20 2023, 20:28

valerio.bozzolan accepted this revision.May 20 2023, 20:29

valerio.bozzolan edited the summary of this revision. (Show Details)

Closed by commit rP10fae71a042f: Fix PHP 8.1 "explode(null)" exception which blocks rendering "Provide Multi…. · Explain WhyMay 21 2023, 05:57

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rP10fae71a042f: Fix PHP 8.1 "explode(null)" exception which blocks rendering "Provide Multi….

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

auth/

storage/

PhabricatorAuthChallenge.php

2 lines

O1: Blessed Committers

Diff 771

src/applications/auth/storage/PhabricatorAuthChallenge.php

Loading...