Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Login form creating Google crawl error #5345

Closed
Ruud68 opened this issue Dec 8, 2014 · 7 comments
Closed

Login form creating Google crawl error #5345

Ruud68 opened this issue Dec 8, 2014 · 7 comments

Comments

@Ruud68
Copy link
Contributor

Ruud68 commented Dec 8, 2014

Hi,
been investigating some Google crawl errors and ran into one created on all pages where the login module is active and where the base url + first direcotry is not working.
When I am e.g. on this page: https://www.lerendeleiders.nl/blog-layout/63-Ruud
it will put this code in the resulting html file:

<div class="overflow">
                                                    <form action="https://www.lerendeleiders.nl/blog-layout" method="post" id="login-form" class="form-inline">
        <div class="userdata">

the culprit is in the url specified in the form action. As you can see it is truncated to the first directory only. the url specified here throws a 403 thus resulting in a crawl error for Google.

This is impacting our Google Ranking as Google wants me to solve these errors which I can't :(

Steps to reproduce the issue

goto url https://www.lerendeleiders.nl/user-profile-layout/63-Ruud
form action = url that is not working

Expected result

form action = url that is working

Actual result

specified above

System information (as much as possible)

System Information
Setting Value

PHP Built On Linux linux76.webawere.nl 2.6.32-458.23.2.lve1.2.48.el6.x86_64 #1 SMP Fri Jan 3 11:26:27 EST 2014 x86_64
Database Version 5.1.73
Database Collation utf8_general_ci
PHP Version 5.3.29
Web Server Apache/2
WebServer to PHP Interface litespeed
Joomla! Version Joomla! 3.3.6 Stable [ Ember ] 01-October-2014 02:00 GMT
Joomla! Platform Version Joomla Platform 13.1.0 Stable [ Curiosity ] 24-Apr-2013 00:00 GMT
User Agent Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:34.0) Gecko/20100101 Firefox/34.0

Additional comments

This is impacting our Google Ranking as Google wants me to solve these errors which I can't :(

@infograf768
Copy link
Member

When using the core login module, the core helper code picks the url from the menu. If the page displayed can't be fetched through a direct menu item (including a submenu) then it the mother menu url which is displayed.
If the login is a menu item and not a module, no issue

@Ruud68
Copy link
Contributor Author

Ruud68 commented Dec 8, 2014

hi, thanks @infograf768 for looking into this
2 questions:

  1. Why is the mother url displayed and not the full url?
  2. what is this url used for?

@Kubik-Rubik
Copy link
Member

I think that we could work with the current URL here as the action parameter. Let me do this change, will provide a PR for it.

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5345.

@Ruud68
Copy link
Contributor Author

Ruud68 commented Dec 8, 2014

Hi @Kubik-Rubik Thanks for looking into this, this was just an example but I have more of these menu items that are a place holder and need a parameter to work correctly. So this impacts my SEO 'heavily' in a bad way, a fix would make Google and me very happy :)
If you need testing, please let me know!
thanks!
Ruud.

@Kubik-Rubik
Copy link
Member

@Ruud68 Please test this PR: #5346

@jissues-bot
Copy link

Set to "closed" on behalf of @zero-24 by The JTracker Application at issues.joomla.org/joomla-cms/5345

@zero-24
Copy link
Contributor

zero-24 commented Dec 8, 2014

Closing as w have a PR by @Kubik-Rubik here: #5346 Thanks

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5345.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants