Fix strlen(null) PHP 8.1 error when pushing into a fresh repository
ClosedPublic
Actions

Authored by Sten on Jul 28 2023, 13:28.

Tags

None

Referenced Files

	F3783212: D25361.1745905609.diff
	Mon, Apr 28, 05:46

	F3766771: D25361.1745846622.diff
	Sun, Apr 27, 13:23

	F3647319: D25361.1745402616.diff
	Tue, Apr 22, 10:03

	F3605281: D25361.1745260538.diff
	Sun, Apr 20, 18:35

	F3545097: D25361.1744898629.diff
	Wed, Apr 16, 14:03

	F3519688: D25361.1744813674.diff
	Tue, Apr 15, 14:27

	F3390366: D25361.1744466267.diff
	Fri, Apr 11, 13:57

	F3369918: D25361.1744216884.diff
	Tue, Apr 8, 16:41

Subscribers

Details

Reviewers

speck

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15569: git push into a fresh repository fails with PHP 8.1 strlen(null) error

Commits

rP189a5d60478e: Fix strlen(null) PHP 8.1 error when pushing into a fresh repository

Summary

Doing a git push into a fresh repository over https fails with a PHP 8.1 strlen(null) error.

This fixes it.

Fixes T15569

Test Plan

Create an empty repository on a PHP 8.1 Phorge instance
Clone it
Add some data
git push

Diff Detail

Repository

rP Phorge

Branch

gitPush

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 695
Build 695: arc lint + arc unit

Event Timeline

Sten created this revision.Jul 28 2023, 13:28

Owners added a reviewer: O1: Blessed Committers.Jul 28 2023, 13:28

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

Harbormaster completed remote builds in B695: Diff 1163.Jul 28 2023, 13:28

Sten requested review of this revision.Jul 28 2023, 13:28

speck accepted this revision.Jul 28 2023, 15:23

This revision is now accepted and ready to land.Jul 28 2023, 15:23

Closed by commit rP189a5d60478e: Fix strlen(null) PHP 8.1 error when pushing into a fresh repository. · Explain WhyJul 28 2023, 17:23

This revision was automatically updated to reflect the committed changes.

Sten added a commit: rP189a5d60478e: Fix strlen(null) PHP 8.1 error when pushing into a fresh repository.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

diffusion/

engine/

DiffusionCommitHookEngine.php

2 lines

O1: Blessed Committers

Diff 1163

View Options

src/applications/diffusion/engine/DiffusionCommitHookEngine.php

Fix strlen(null) PHP 8.1 error when pushing into a fresh repositoryClosedPublicActions