Project

General

Profile

Actions

Bug #5671

closed

Stack trace due to audit table field being too short

Added by Brad DeMorrow over 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Audit Log
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

I began receiving the following trace while trying to update a parameter override.

ActiveRecord::StatementInvalid
PG::StringDataRightTruncation: ERROR: value too long for type character varying(255) : INSERT INTO "audits" ("action", "associated_id", "associated_name", "associated_type", "auditable_id", "auditable_name", "auditable_type", "audited_changes", "comment", "created_at", "remote_address", "user_id", "user_type", "username", "version") VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15) RETURNING "id"
app/models/concerns/host_common.rb:46:in `block in lookup_values_attributes='
app/models/concerns/host_common.rb:39:in `each_value'
app/models/concerns/host_common.rb:39:in `lookup_values_attributes='
app/controllers/hostgroups_controller.rb:64:in `update'
app/models/concerns/foreman/thread_session.rb:33:in `clear_thread'
lib/middleware/catch_json_parse_errors.rb:9:in `call'

The parameter I'm updating contained usernames and their sha512 hashes.


Related issues 6 (0 open6 closed)

Related to Foreman - Bug #1902: Smart Parameters aren't audit loggedClosedDominic Cleal10/17/2012Actions
Has duplicate Foreman - Bug #5721: Updating host group parameter in 1.5.0 give value too long for type character varying(255)Duplicate05/14/2014Actions
Has duplicate Foreman - Bug #5762: Multiple matchers on smart variableDuplicate05/16/2014Actions
Has duplicate Foreman - Bug #5807: PGError: "value too long for type character varying(255)" for INSERT INTO "audits" when overriding "smart variable" in hostgroup with >255 characters value.Duplicate05/20/2014Actions
Has duplicate Foreman - Bug #6087: input into audit goes wrong Duplicate06/06/2014Actions
Has duplicate Foreman - Bug #6280: crash adding a smart variable matcherDuplicate06/18/2014Actions
Actions #1

Updated by Brad DeMorrow over 10 years ago


sudo -u foreman psql -c '\d audits'
                                        Table "public.audits" 
     Column      |            Type             |                      Modifiers
-----------------+-----------------------------+-----------------------------------------------------
 id              | integer                     | not null default nextval('audits_id_seq'::regclass)
 auditable_id    | integer                     |
 auditable_type  | character varying(255)      |
 user_id         | integer                     |
 user_type       | character varying(255)      |
 username        | character varying(255)      |
 action          | character varying(255)      |
 audited_changes | text                        |
 version         | integer                     | default 0
 comment         | character varying(255)      |
 associated_id   | integer                     |
 associated_type | character varying(255)      |
 created_at      | timestamp without time zone |
 remote_address  | character varying(255)      |
 auditable_name  | character varying(255)      |
 associated_name | character varying(255)      |
Indexes:
    "audits_pkey" PRIMARY KEY, btree (id)
    "associated_index" btree (associated_id, associated_type)
    "auditable_index" btree (auditable_id, auditable_type)
    "auditable_parent_index" btree (associated_id, associated_type)
    "index_audits_on_created_at" btree (created_at)
    "index_audits_on_id" btree (id)
    "user_index" btree (user_id, user_type)
Foreign-key constraints:
    "audits_user_id_fk" FOREIGN KEY (user_id) REFERENCES users(id)
Actions #2

Updated by Dominic Cleal over 10 years ago

  • Related to Bug #1902: Smart Parameters aren't audit logged added
Actions #3

Updated by Dominic Cleal over 10 years ago

  • Category set to Audit Log
  • Translation missing: en.field_release set to 16
Actions #4

Updated by Brad DeMorrow over 10 years ago

After a little bit of digging, it looks like auditable_name is the culprit.

Actions #5

Updated by Dominic Cleal over 10 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Dominic Cleal
  • Target version set to 1.8.3
Actions #6

Updated by Dominic Cleal over 10 years ago

  • Has duplicate Bug #5721: Updating host group parameter in 1.5.0 give value too long for type character varying(255) added
Actions #7

Updated by Dominic Cleal over 10 years ago

  • Has duplicate Bug #5762: Multiple matchers on smart variable added
Actions #8

Updated by Anonymous over 10 years ago

  • Target version changed from 1.8.3 to 1.8.2
Actions #9

Updated by Dominic Cleal over 10 years ago

  • Has duplicate Bug #5807: PGError: "value too long for type character varying(255)" for INSERT INTO "audits" when overriding "smart variable" in hostgroup with >255 characters value. added
Actions #10

Updated by Dominic Cleal over 10 years ago

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

Updated by Dominic Cleal over 10 years ago

  • Has duplicate Bug #6087: input into audit goes wrong added
Actions #12

Updated by Bryan Kearney over 10 years ago

  • Bugzilla link set to https://bugzilla.redhat.com/show_bug.cgi?id=1107701
Actions #13

Updated by Dominic Cleal over 10 years ago

  • Has duplicate Bug #6280: crash adding a smart variable matcher added
Actions

Also available in: Atom PDF