Project

General

Profile

Bug #24057

Email subject prefix accepts long strings

Added by Aditi Puntambekar 3 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Low
Category:
Settings
Target version:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Description of problem:

Most string fields are limited to 255 chars but that is not the case of email subject prefix

Version-Release number of selected component (if applicable):

Satellite 6.3 snap 14

How reproducible:

Always

Steps to Reproduce:

hammer> settings set --name=email_subject_prefix --value="pppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppp"

Actual results:

Email subject prefix is updated with a string containing 1000 chars.

Expected results:

Error msg limiting the field to 255 chars.

Additional info:

Associated revisions

Revision 87e23352 (diff)
Added by Aditi Puntambekar 3 months ago

Fixes #24057 - Email subject prefix accepts long strings (#5742)

Revision 151ecb5d (diff)
Added by Lukas Zapletal 3 months ago

Refs #24057 - fixed broken email test

History

#1 Updated by Aditi Puntambekar 3 months ago

  • Triaged set to No
  • Pull request https://github.com/theforeman/foreman/pull/5742 added

#2 Updated by The Foreman Bot 3 months ago

  • Status changed from New to Ready For Testing

#3 Updated by Aditi Puntambekar 3 months ago

  • Category set to Settings

#4 Updated by Aditi Puntambekar 3 months ago

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

#5 Updated by The Foreman Bot 3 months ago

  • Pull request https://github.com/theforeman/foreman/pull/5757 added

#6 Updated by Marek Hulán 3 months ago

  • Legacy Backlogs Release (now unused) set to 353

Also available in: Atom PDF