| Anonymous | Login | 2024-11-21 05:23 MST |
| Main | My View | View Issues | Change Log | Roadmap | Repositories |
| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||
| 0000066 | ClearOS | app-settings - System Settings | public | 2010-03-27 13:01 | 2010-06-01 21:02 | ||||
| Reporter | bchambers | ||||||||
| Assigned To | user2 | ||||||||
| Priority | normal | Severity | minor | Reproducibility | always | ||||
| Status | closed | Resolution | fixed | ||||||
| Platform | OS | OS Version | |||||||
| Product Version | 5.1 | ||||||||
| Target Version | 5.2 | Fixed in Version | 5.2 | ||||||
| Summary | 0000066: Allow 'localhost' as a valid hostname for SMTP hostname field | ||||||||
| Description | Mail notification uses the Network class's "IsValidHostname" function to validate input of the form variables. This function checks for the presence of a 'period', and throws a validation error if the hostname does not contain at least one: /var/webconfig/api/Hostname.class.php if (substr_count($hostname, ".") == 0) { $errmsg = NETWORK_LANG_ERRMSG_HOSTNAME_MUST_HAVE_A_PERIOD; $this->AddValidationError($errmsg, __METHOD__, __LINE__); return false; } The entry 'localhost' is valid...but in the Network class, this may want to be more strict. For the Mail Notification, a check for 'localhost' before the network class validation routine could be a solution: if ($host == 'localhost') return true; $network = new Network(); if (! $network->IsValidHostname($host)) throw new ValidationException(implode($network->GetValidationErrors(true))); return true; | ||||||||
| Additional Information | There are lots of 'workarounds' to get around this bug until it is fixed. Use the local IP (192.168.1.1, for example) Use the local hostname (clear.lan, for example) Use the external hostname (myclearbox.poweredbyclear.com, for example) Create an entry in the hosts file. | ||||||||
| Tags | No tags attached. | ||||||||
| Attached Files | |||||||||
 Relationships |
|
 Relationships |
|
Notes |
|
|
(0000063) timb80 (developer) 2010-03-27 13:40 |
Another alternative? /var/webconfig/api/Network.class.php line 300 Function "IsValidHostname($hostname)" Change Code: if (substr_count($hostname, ".") == 0 ) { TO if (substr_count($hostname, ".") == 0 && $hostname!='localhost') { |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2010-03-27 13:01 | bchambers | New Issue | |
| 2010-03-27 13:40 | timb80 | Note Added: 0000063 | |
| 2010-03-28 20:05 | user2 | Status | new => assigned |
| 2010-03-28 20:05 | user2 | Assigned To | => user2 |
| 2010-03-28 20:07 | user2 | Target Version | => 5.2 |
| 2010-03-30 18:03 | user2 | Checkin | |
| 2010-03-30 18:03 | user2 | Note Added: 0000072 | |
| 2010-03-30 18:03 | user2 | Status | assigned => resolved |
| 2010-03-30 18:03 | user2 | Resolution | open => fixed |
| 2010-05-19 11:30 | user2 | Fixed in Version | => 5.2 |
| 2010-06-01 21:02 | user2 | Status | resolved => closed |
| 2010-08-18 15:05 | user2 | Note Deleted: 0000072 | |
|
Issue History |