• I installed a fresh version of Drupal8-alpha8 via the standard web installer wizard.
  • A few moments later I received an e-mail notifying me that updates were available, however the link is 404/not found.
  • The e-mail URL is pointing to example.com/core/admin/reports/updates, but the correct page is example.com/admin/reports/updates
  • The updates page URL is correct when navigating to it through the Admin menu - [Reports] > Available Updates, it seems to be only the e-mail notice updates URL that is incorrect.

Your version of Drupal is no longer supported. Upgrading is strongly
recommended!

See the available updates page for more information:
http://drupal8-alpha8.sandbox.dev/core/admin/reports/updates

Your site is currently configured to send these emails only when security
updates are available. To get notified for any available updates,
http://drupal8-alpha8.sandbox.dev/core/admin/reports/updates/settings.

Updates - 404 page not found

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RowboTony’s picture

Issue summary: View changes
rahulbile’s picture

Version: 8.0-alpha8 » 8.x-dev
Status: Active » Needs review
FileSize
1.13 KB

Here is the initial patch against 8.x which tries to make sure the base_url is properly set when generating url through generateFromPath in Urlgenerator.php.

Status: Needs review » Needs work

The last submitted patch, 2: 2188167-2-core-install-url.patch, failed testing.

rahulbile’s picture

Status: Needs work » Needs review
FileSize
1.44 KB

Updated patch to consider request URI.

RowboTony’s picture

Hello, thank you for the patch. I downloaded a fresh install of Drupal8 alpha8, applied the patch, and proceeded to install via the web installer. Shortly after the install completed I received an e-mail with the correct links to the updates:

Your version of Drupal is no longer supported. Upgrading is strongly
recommended!

See the available updates page for more information:
http://drupal8-alpha8.groffweb.dev/admin/reports/updates

Your site is currently configured to send these emails only when security
updates are available. To get notified for any available updates,
http://drupal8-alpha8.groffweb.dev/admin/reports/updates/settings.

So the patch does fix the email notification, however I did find a regression issue that seems to be related to this patch - after the web installer completes, the page says:

Drupal installation complete

Congratulations, you installed Drupal!

Visit your new site

Unfortunately, when this patch is applied, it breaks this link. The link now points to "//" after the patch is applied.

<a href="//">Visit your new site</a>

I did another fresh install of drupal-8.0-alpha8.tar.gz, I did not apply the patch this time, proceeded through the web installer, and that link is ok and correctly pointing to my full site URL. So, all things considered, with this patch being the only thing applied, it does break the "Visit your new site" URL.

visit your new site link is broken
visit your new site link is broken

rahulbile’s picture

That was a a good catch @drupleg , attached is the updated patch to crack this.

jhedstrom’s picture

Status: Needs review » Closed (fixed)

I think this was fixed with #2340251: Remove most remaining url() calls.