Skip to content

lg-15077 update sensitivetag #11514

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
samathad2023 merged 2 commits into from
Nov 18, 2024
Merged

lg-15077 update sensitivetag #11514

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b589c4a
lg-15077 update sensitivetag
samathad2023 Nov 15, 2024
c4dae81
migration comment append
samathad2023 Nov 15, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion app/jobs/data_warehouse/daily_sensitive_column_job.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@ def perform(timestamp)
end

def fetch_columns
tables = ActiveRecord::Base.connection.tables - %w[schema_migrations ar_internal_metadata]
tables = ActiveRecord::Base.connection.tables - %w[schema_migrations ar_internal_metadata
awsdms_ddl_audit]

sensitive_hash = []
insensitive_hash = []
Expand Down
7 changes: 7 additions & 0 deletions db/primary_migrate/20241115215510_update_sensitive_columns_comments.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
class UpdateSensitiveColumnsComments < ActiveRecord::Migration[7.2]
def change
#change columns to sensitive=true
change_column_comment :phone_number_opt_outs, :phone_fingerprint, from: "sensitive=false", to: "sensitive=true"
change_column_comment :in_person_enrollments, :selected_location_details, from: "The location details of the Post Office the user selected (including title, address, hours of operation) sensitive=false", to: "The location details of the Post Office the user selected (including title, address, hours of operation) sensitive=true"
end
end
6 changes: 3 additions & 3 deletions db/schema.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema[7.2].define(version: 2024_10_29_152408) do
ActiveRecord::Schema[7.2].define(version: 2024_11_15_215510) do
# These are extensions that must be enabled in order to support this database
enable_extension "citext"
enable_extension "pg_stat_statements"
Expand Down Expand Up @@ -312,7 +312,7 @@
t.datetime "created_at", null: false, comment: "sensitive=false"
t.datetime "updated_at", null: false, comment: "sensitive=false"
t.boolean "current_address_matches_id", comment: "True if the user indicates that their current address matches the address on the ID they're bringing to the Post Office. sensitive=false"
t.jsonb "selected_location_details", comment: "The location details of the Post Office the user selected (including title, address, hours of operation) sensitive=false"
t.jsonb "selected_location_details", comment: "The location details of the Post Office the user selected (including title, address, hours of operation) sensitive=true"
t.string "unique_id", comment: "Unique ID to use with the USPS service sensitive=false"
t.datetime "enrollment_established_at", comment: "When the enrollment was successfully established sensitive=false"
t.string "issuer", comment: "Issuer associated with the enrollment at time of creation sensitive=false"
Expand Down Expand Up @@ -417,7 +417,7 @@

create_table "phone_number_opt_outs", force: :cascade do |t|
t.string "encrypted_phone", comment: "sensitive=true"
t.string "phone_fingerprint", null: false, comment: "sensitive=false"
t.string "phone_fingerprint", null: false, comment: "sensitive=true"
t.string "uuid", comment: "sensitive=false"
t.datetime "created_at", null: false, comment: "sensitive=false"
t.datetime "updated_at", null: false, comment: "sensitive=false"
Expand Down