Project

General

Profile

Feature #15370

Additional validations for email and trusted hosts settings

Added by Ondřej Pražák about 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Settings
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

The settings for administrator and email_reply_address do not validate email format and trusted_puppetmaster_hosts allow any values as long as input is an array.


Related issues

Related to Foreman - Bug #6924: UI should throw proper validation error while updating some params with any random value(like utf-8 or -ve values)Closed2014-08-05

Associated revisions

Revision 643ca12c (diff)
Added by Ondřej Pražák about 4 years ago

Fixes #15370 - Validate settings values

Adds validations for administrator, email_reply_address
and trusted_puppetmaster_hosts settings

History

#1 Updated by Ondřej Pražák about 4 years ago

  • Related to Bug #6924: UI should throw proper validation error while updating some params with any random value(like utf-8 or -ve values) added

#2 Updated by Dominic Cleal about 4 years ago

  • Tracker changed from Bug to Feature
  • Subject changed from Additional validations for Settings to Additional validations for email and trusted hosts settings
  • Category set to Settings

#3 Updated by The Foreman Bot about 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ondřej Pražák
  • Pull request https://github.com/theforeman/foreman/pull/3583 added

#4 Updated by Ondřej Pražák about 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal about 4 years ago

  • Legacy Backlogs Release (now unused) set to 160

Also available in: Atom PDF