I'm using http://localhost/csp-torchbox/web/core/install.php/admin/config/content/... to install my site, and the imported config includes honeypot module.

The install message shown by hook_install() contains a broken URL:

> http://localhost/my-site/core/install.php/admin/config/content/honeypot

I would suggest not showing the message at all when the module is installed as part of a site installation.

CommentFileSizeAuthor
#4 2803839-4-use-route-in-install-message.patch775 bytesTR
#3 2803839-3-use-route-in-install-message.patch775 bytesTR
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

imperator_99’s picture

imperator_99 CreditAttribution: imperator_99 commented

I'm getting the same error using version 8.x-1.27. The problem seems to be in using 'base://' prefix for the URL, which seems to pick up the most recent URL (in this case, /core/install.php).

TR’s picture

TR CreditAttribution: TR commented
Version: 8.x-1.x-dev » 2.0.x-dev
Status: Active » Needs review
FileSize
2803839-3-use-route-in-install-message.patch775 bytes

Well really the right way to do it in D8/D9 is to use the route instead of trying to construct a complete absolute path.

Can you test this to see if it fixes your problem?

TR’s picture

TR CreditAttribution: TR commented
FileSize
2803839-4-use-route-in-install-message.patch775 bytes

Sorry, I made a stupid syntax error. Here's the correct patch.

  • TR committed 37d8e3a on 2.0.x 
    Issue #2803839 by TR: URL in message is broken when honeypot is...
TR’s picture

TR CreditAttribution: TR commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.