Paths

Table of Contentst

in robots.txt
ClosedPublic
Actions

Authored by epriestley on Apr 6 2017, 10:53 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jan 3, 8:33 AM

	Unknown Object (File)
	Tue, Dec 31, 10:15 AM

	Unknown Object (File)
	Fri, Dec 27, 7:16 PM

	Unknown Object (File)
	Fri, Dec 27, 7:12 PM

	Unknown Object (File)
	Fri, Dec 27, 7:12 PM

	Unknown Object (File)
	Fri, Dec 27, 6:50 PM

	Unknown Object (File)
	Sun, Dec 22, 5:43 PM

	Unknown Object (File)
	Fri, Dec 20, 6:54 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T4245: Make repository URIs slightly prettier

Commits

rP9856802ba22e: Disallow /source/ in robots.txt

Summary

Ref T4245. We disallow /diffusion/ in robots.txt already because indexers tend to get lost blaming every line of every file throughout history, but didn't update the list for the /source/ alias. Update it.

Test Plan

Visited /robots.txt locally.

Diff Detail

Repository

rP Phabricator

Branch

robot1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 16346
Build 21737: Run Core Tests
Build 21736: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 6 2017, 10:53 PM

Harbormaster completed remote builds in B16346: Diff 42418.Apr 6 2017, 10:54 PM

chad accepted this revision.Apr 6 2017, 11:19 PM

This revision is now accepted and ready to land.Apr 6 2017, 11:19 PM

Closed by commit rP9856802ba22e: Disallow /source/ in robots.txt (authored by epriestley, committed by epriestley). · Explain WhyApr 6 2017, 11:28 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

system/

controller/

PhabricatorRobotsController.php

1 line

Diff 42418

View Options

src/applications/system/controller/PhabricatorRobotsController.php

Disallow /source/ in robots.txtClosedPublicActions