Home GnuPG

Remove "url" from Elasticsearch index
6d8167503266Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Remove "url" from Elasticsearch index

Summary:
Ref T12450. This was added a very very long time ago (D2298).

I don't want to put this in the upstream index anymore because I don't want to encourage third parties to develop software which reads the index directly. Reading the index directly is a big skeleton key which bypasses policy checks.

This was added before much of the policy model existed, when that wasn't as much of a concern. On a tecnhnical note, this also doesn't update when phabricator.base-uri changes.

This can be written as a search index extension if an install relies on it for some bizarre reason, although none should and I'm unaware of any actual use cases in the wild for it, even at Facebook.

Test Plan: Indexed some random stuff into ElasticSearch.

Reviewers: chad, 20after4

Reviewed By: chad

Maniphest Tasks: T12450

Differential Revision: https://secure.phabricator.com/D17600

Details

Provenance
epriestley <git@epriestley.com>Authored on Apr 2 2017, 6:59 PM
Parents
rPHAB287e708c4d3e: Adjust and wordsmith Search documentation
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB6d8167503266: Remove "url" from Elasticsearch index (authored by epriestley <git@epriestley.com>).Apr 2 2017, 10:26 PM