Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The highest port number is 65535 instead of 65536.
http://stackoverflow.com/questions/113224/what-is-the-largest-tcp-ip-net...
The validation is wrong in the \Core\Database\Install\Tasks::getFormOptions()
$form['advanced_options']['port'] = array(...);
Comment | File | Size | Author |
---|---|---|---|
#1 | drupal-maximum-port-number-2228801-01-d8.patch | 531 bytes | Sweetchuck |
Comments
Comment #1
SweetchuckComment #2
danblack CreditAttribution: danblack commentednice and simple. For a newbie, why doesn't this have validateDatabaseSettings validating the port number like #1938224?
Comment #3
Sweetchuck@danblack #2 We do not need validate manually the port number because the new
#type = 'number'
form element type do the job thanks to#min
and#max
properties.Comment #4
danblack CreditAttribution: danblack commentedgood! :-)
Comment #5
webchickOops, nice catch. :)
Committed and pushed to 8.x. Thanks!