Page MenuHomePhabricator

Don't show client IP in push logs unless viewer can edit the repository
ClosedPublic

Authored by epriestley on Dec 5 2013, 11:32 PM.
Tags
None
Referenced Files
F13816784: D7722.id17442.diff
Thu, Sep 19, 5:46 PM
F13816781: D7722.id17437.diff
Thu, Sep 19, 5:42 PM
F13814160: D7722.diff
Thu, Sep 19, 3:57 AM
Unknown Object (File)
Sun, Aug 25, 10:27 AM
Unknown Object (File)
Sun, Aug 25, 7:58 AM
Unknown Object (File)
Sun, Aug 25, 7:55 AM
Unknown Object (File)
Sun, Aug 25, 7:54 AM
Unknown Object (File)
Sun, Aug 25, 7:53 AM
Subscribers

Details

Summary

This locks push logs down a little bit and makes them slightly more administrative. Primarily, don't show IPs to googlebot, etc.

Test Plan

Viewed push logs as edit and non-edit users.

Diff Detail

Branch
pushlogip
Lint
Lint Passed
Unit
No Test Coverage