Page MenuHomePhabricator

Remove all references to the Conduit ConnectionLog
ClosedPublic

Authored by epriestley on Dec 14 2015, 8:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 1:41 AM
Unknown Object (File)
Fri, Apr 19, 3:56 PM
Unknown Object (File)
Fri, Apr 12, 10:15 PM
Unknown Object (File)
Thu, Apr 11, 9:36 AM
Unknown Object (File)
Mon, Apr 8, 10:25 AM
Unknown Object (File)
Sun, Apr 7, 3:01 AM
Unknown Object (File)
Sat, Mar 30, 1:03 AM
Unknown Object (File)
Mar 22 2024, 12:47 PM
Subscribers
None

Details

Summary

Ref T5955, T9980, T9982.

We currently store two types of Conduit logs: connection logs and method logs.

Originally, Conduit worked like web logins: you'd call conduit.connect and then get a session back. This approach still works, but new clients don't use it and it will probably stop working eventually after T5955 is further along.

There was no real reason for things to work like this and no other API in the world does, I think it was just slightly easier to implement back in 2011.

This table was used to group up related calls in a UI long ago, I think, but that got deleted at some point. In any case, it serves no purpose in modern Phabricator.

Test Plan

grep

Diff Detail

Repository
rP Phabricator
Branch
clog3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 9559
Build 11423: Run Core Tests
Build 11422: arc lint + arc unit

Event Timeline

chad edited edge metadata.
This revision is now accepted and ready to land.Dec 14 2015, 8:55 PM
This revision was automatically updated to reflect the committed changes.