Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#23859 closed defect (bug) (duplicate)

Logout Url getting indexed

Reported by: unsalkorkmaz's profile unsalkorkmaz Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Toolbar Keywords:
Focuses: Cc:

Description

Problem line: http://core.trac.wordpress.org/browser/tags/3.5.1/wp-includes/class-wp-admin-bar.php#L357

<a class="screen-reader-shortcut" href="<?php echo esc_url( wp_logout_url() ); ?>"><?php _e('Log Out'); ?></a>

It gets indexed by google and probably other search engines too. (see: attachment)

Possible solution: Maybe adding rel="nofollow" will fix problem

Attachments (1)

problem.png (42.0 KB) - added by unsalkorkmaz 10 years ago.
google index logout urls

Download all attachments as: .zip

Change History (5)

@unsalkorkmaz
10 years ago

google index logout urls

#1 @johnbillion
10 years ago

  • Keywords reporter-feedback 2nd-opinion added

We already call wp_no_robots() on the login screen: http://core.trac.wordpress.org/browser/tags/3.5.1/wp-login.php#L44, which outputs the following in the header:

<meta name='robots' content='noindex,nofollow' />

Not sure what to suggest here. Are you running any "SEO" plugins that might be interfering?

#2 @unsalkorkmaz
10 years ago

This is displaying in wp_admin_bar and when you show wp_admin_bar to non-logged-in users, it shows on every page

#3 @toscho
10 years ago

  • Cc info@… added
  • Keywords close added

A properly written log-out URL will be visible to logged in users only. So this problem can never happen.

#4 @SergeyBiryukov
10 years ago

  • Component changed from General to Toolbar
  • Keywords reporter-feedback 2nd-opinion close removed
  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

#23178 has a patch.

Note: See TracTickets for help on using tickets.