Page MenuHomePhabricator
Differential D13432

Fix an issue with handling of `null` feed URIs
ClosedPublic
Actions

Authored by epriestley on Jun 25 2015, 1:12 PM.
Tags
None
Referenced Files
F19574831: D13432.id.diff
Sun, Feb 1, 6:45 AM
F19538269: D13432.diff
Wed, Jan 21, 3:10 PM
F19535432: D13432.id.diff
Wed, Jan 21, 6:15 AM
F19526629: D13432.diff
Sun, Jan 18, 6:45 PM
F19303841: D13432.id32535.diff
Dec 24 2025, 5:55 AM
F19153178: D13432.diff
Dec 11 2025, 8:05 AM
F19129369: D13432.id.diff
Dec 10 2025, 4:29 AM
F19017671: D13432.id32519.diff
Nov 23 2025, 8:43 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP1192f309b016: Fix an issue with handling of `null` feed URIs
Summary

Ref T8658. Caught this in the logs. This value may be set to null. Handle that gracefully.

Test Plan

Will check logs.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 32519.Jun 25 2015, 1:12 PM
epriestley retitled this revision from to Fix an issue with handling of `null` feed URIs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: Restricted Maniphest Task.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 25 2015, 1:12 PM
btrahan accepted this revision.Jun 25 2015, 5:00 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 25 2015, 5:00 PM
Closed by commit rP1192f309b016: Fix an issue with handling of `null` feed URIs (authored by epriestley, committed by epriestley). · Explain WhyJun 25 2015, 5:01 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
feed/
worker/
17 lines

Diff 32535

View Options

src/applications/feed/worker/FeedPublisherWorker.php

Loading...
Phabricator · Built by Phacility