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
F15180532: D13432.id32535.diff
Wed, Feb 19, 8:46 PM
Unknown Object (File)
Tue, Feb 18, 4:31 PM
Unknown Object (File)
Sat, Feb 8, 3:21 PM
Unknown Object (File)
Sat, Feb 8, 3:20 PM
Unknown Object (File)
Sat, Feb 8, 3:20 PM
Unknown Object (File)
Sat, Feb 8, 3:19 PM
Unknown Object (File)
Sat, Feb 8, 3:17 AM
Unknown Object (File)
Tue, Jan 28, 11:04 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