Registration and Change password urls have less size available than I needed.
This supposed a problem in my environment, but I thought that maybe I was using too long urls. However, I saw that the other url fields are longer than these, so I attach a patch for making them all equal.

CommentFileSizeAuthor
#1 urls-size-1131878-1.patch851 bytespenyaskito
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito commented
FileSize
urls-size-1131878-1.patch851 bytes

Attached a patch for 6.x-3.x, the branch that I used.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito commented
Status: Active » Needs review

This is already in 7.x-1.x, so no patch needed. Please review 6.x-3.x patch.

bfroehle’s picture

bfroehle CreditAttribution: bfroehle commented

I thought the default #size is 60 and default #maxlength is 128. Is that not sufficient?

bfroehle’s picture

bfroehle CreditAttribution: bfroehle commented
Status: Needs review » Fixed

Nonetheless, I've committed your patch which sets '#maxlength' to 255. In doing so, I also changed the maxlength of paths and hostnames to 255 to match the defaults in Drupal core, and reduced the maxlength of the 'CAS port' to 5 (since the maximum port number is 65536).

Thanks for your contribution! Committed to 6.x-3.x and 7.x-1.x.

Status: Fixed » Closed (fixed)

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

  • Commit 65ff879 on 7.x-1.x, 8.x-1.x authored by penyaskito, committed by bfroehle: 
    Issue #1131878 - Registration and Change password urls have different...