Page MenuHomePhorge

Fix PHP 8.1 "strlen(null)" exception from ArcanistRefView which blocks "arc look remotes"
ClosedPublic

Authored by valerio.bozzolan on Mon, May 8, 17:04.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 31, 06:40
Unknown Object (File)
Sat, May 27, 03:14
Unknown Object (File)
Sat, May 27, 00:54
Unknown Object (File)
Sun, May 21, 13:25

Details

Summary

This change fixes the command arc look remotes for PHP 8.1.

Without this change, the null value bubbles up to PhutilUTF8StringTruncator, reaching a strlen().

This control probably does not need to be done at this low level inside PhutilUTF8StringTruncator,
but it is right to be at this high level from the caller in ArcanistRefView.

Closes T15368

Test Plan
  • run "arc look remotes"
  • still works in "old PHP" like 7.4
  • start to work in recent PHP 8.1+

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

src/ref/ArcanistRefView.php
103

↑ here the problematic area - this should not receive null

Hi @Matthew :D super-safe thing here. Do you love this?

I think @speck maybe has better approaches here (do you know arc look? it's lovely)

This revision is now accepted and ready to land.Thu, Jun 1, 15:38