Opened 16 years ago
Closed 16 years ago
#10747 closed defect (bug) (duplicate)
comment_agent field is too small
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Severity: | normal | Version: | 2.8.4 |
| Component: | Comments | Keywords: | needs-patch |
| Focuses: | Cc: |
Description
The comment_agent field in comments table is too small (255).
This causes an error if user agent field is larger than 255 - the comment is not inserted, and no message is displayed.
making this field 500 length fixed this problem
Change History (7)
#1
@
16 years ago
- Component changed from General to Comments
- Keywords dev-feedback added; comments removed
#3
@
16 years ago
I am not sure that this can wait given the coming release of Windows 7 next month. After I upgraded from windows Vista to Windows 7 I discovered that my user agent was too long. Further investigation revealed that many installations of Internet Explorer on computers that have been in use for a period of time and have multiple versions of the .NET CLR, etc. installed have user agent strings which exceed 255 when the Windows 7 specific strings were added after the upgrade. (The user agent sting is shorter and not problematical in fresh installations of Windows 7.) The likely hood of this happening becomes even greater with the introduction of tools like the Google Chrome Frame which add to the user agent string.
See #10389 - Truncate the field in order to prevent errors (not inserting).
Not sure increasing the length would be advisable however..
dev-feedback pending review of increasing the length or not. (If meta comes in, Storing such items in commentmeta would be better suited IMO)