The custom URL length should have either no max length or one much higher, particularly for using tokens.

For example, one of my more lengthy tokens ends up as [submission:values:personal:occupants_other:occupants_other_1:occupants_other_1_age:nolabel]. The actual value of the token is a two-digit integer, but I cannot use more than two of these tokens in my custom URL.

Another example, I use conditionals to "show" a hidden value for each option of a select field which has a couple dozen options. I want to post the "shown" hidden value for a selection on a PHP "thank you" page, so the URL will have a query string like "q=[submission:values:hidden1:nolabel][submission:values:hidden2:nolabel][submission:values:hidden3:nolabel]..." I have to string together all the hidden values to post the single hidden value that was "shown."

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

Title: Custom URL Length » Increase custom URL length
Category: support » task
Status: Active » Fixed

Totally agreed. This should have been filed under a task/bug/feature. I've increased it to 2048 characters instead of 256.

quicksketch’s picture

Version: 7.x-4.0-alpha4 » 7.x-4.0-alpha6

This will be in the 7.x-4.0-alpha7 release.

Status: Fixed » Closed (fixed)

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

Liam Morland’s picture

Version: 7.x-4.0-alpha6 » 7.x-4.x-dev
Category: Task » Bug report
Issue summary: View changes
Status: Closed (fixed) » Needs review
FileSize
919 bytes

The database still limits the length to 255. The attached patch increases that limit to 2048.

quicksketch’s picture

Thanks Liam, clearly the database size needs to match.

Liam Morland’s picture

Title: Increase custom URL length » Increase redirect_url length
quicksketch’s picture

Status: Needs review » Fixed

Committed. Thanks!

  • Commit f54e435 on 7.x-4.x by quicksketch:
    Issue #1736982 by Liam Morland: Increase redirect_url length in the...

Status: Fixed » Closed (fixed)

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

fenstrat’s picture

Version: 7.x-4.x-dev » 8.x-4.x-dev
Assigned: Unassigned » fenstrat
Status: Closed (fixed) » Patch (to be ported)

Needs porting to 8.x-4.x.

fenstrat’s picture

Version: 8.x-4.x-dev » 7.x-4.x-dev
Assigned: fenstrat » Unassigned
Status: Patch (to be ported) » Fixed

Committed and pushed f54e435 to 8.x-4.x. Thanks!

  • Commit 9daeb05 on 8.x-4.x authored by Liam Morland, committed by fenstrat:
    Issue #1736982 by Liam Morland: Increase redirect_url length in the...

Status: Fixed » Closed (fixed)

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