Page MenuHomePhabricator

Correct Aphlict websocket URI construction after PHP8 compatibility changes
ClosedPublic

Authored by epriestley on Nov 13 2023, 7:00 PM.
Tags
None
Referenced Files
F15449368: D21875.id52184.diff
Fri, Mar 28, 9:45 AM
F15449354: D21875.id.diff
Fri, Mar 28, 9:40 AM
F15438919: D21875.diff
Wed, Mar 26, 4:39 AM
F15430932: D21875.id52184.diff
Mon, Mar 24, 9:45 AM
F15425997: D21875.id.diff
Sun, Mar 23, 6:48 AM
F15424130: D21875.diff
Sat, Mar 22, 8:23 PM
F15383524: D21875.diff
Fri, Mar 14, 5:01 PM
F15377070: D21875.diff
Thu, Mar 13, 7:46 AM
Subscribers
None

Details

Summary

See D21862. Ref T13700. D21862 affected notification server URI generation behavior when a notification server client is configured with:

  1. no "path" argument; and
  2. "cluster.instance" not set.

Condition (1) is not true default, and condition (2) is not true in my local environment, so it was easy for this to slip through the cracks. Apply the change suggested in D21862.

Also fix a couple other string-null issues I caught locally.

Test Plan

Generated notification server URIs under various simulated local conditions (no instance, path set) and everything seemed to be working correctly.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable