https://www.drupal.org/project/cloudflare can be improved

1. the page mentions admin/settings/cloudflare
This should be admin/config/people/cloudflare instead

2. it mentions that the API key can be copied from "support". It'd be better to directly link to https://www.cloudflare.com/a/account/my-account as this is much clearer

Comments

okokokok’s picture

okokokok commented
Title: admin/config/people/cloudflare not admin/settings/cloudflare » improve documentation
Issue summary: View changes
gido’s picture

gido commented

Agree that the project page should be synced with the README.

For example in the Install instruction, replace step 3:
from this:

if (isset($_SERVER['HTTP_CF_CONNECTING_IP'])) {
  $_SERVER['REMOTE_ADDR'] = $_SERVER['HTTP_CF_CONNECTING_IP'];
}

with this:

$conf['cloudflare_cf_connecting_ip'] = TRUE;

with mention to read the README.md about "IP Address detection".

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
commented

Is the following valid for both 1.x & 2.x branches?

$conf['cloudflare_cf_connecting_ip'] = TRUE;
aaronbauman’s picture

aaronbauman
he/him
commented
Version: 7.x-2.x-dev » 8.x-1.x-dev

Can someone point me to any instructions on how to get started?
Everything is out of date.

rosk0’s picture

rosk0
he/him
Primary language Russian
Location Wellington
commented
Version: 8.x-1.x-dev » 2.0.x-dev
Status: Active » Fixed

Fixed via #3591847: Revamp documentation.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.