Skip to content

Adding metadata and expires_at to Secure Files

Darby Frey requested to merge secure-files-metadata into master

What does this MR do and why?

As the first MR to be extracted from this POC !97548 (closed), this MR adds two fields to the ci_secure_files table in the database. One field is a jsonb metadata field, and the other is the expires_at timestamp. These fields will be used to store metadata for file types we are able to parse. For more context, please see this issue #374949 (closed), and the original POC MR.

Migration Output

rake db:migrate:ci
ci: == 20220921174815 AddSecureFilesMetadata: migrating ===========================
ci: -- add_column(:ci_secure_files, :metadata, :jsonb)
ci:    -> 0.0038s
ci: -- add_column(:ci_secure_files, :expires_at, :datetime_with_timezone)
ci:    -> 0.0005s
ci: == 20220921174815 AddSecureFilesMetadata: migrated (0.0048s) ==================

Rollback Output

rake db:rollback:ci
ci: == 20220921174815 AddSecureFilesMetadata: reverting ===========================
ci: -- remove_column(:ci_secure_files, :expires_at, :datetime_with_timezone)
ci:    -> 0.0028s
ci: -- remove_column(:ci_secure_files, :metadata, :jsonb)
ci:    -> 0.0004s
ci: == 20220921174815 AddSecureFilesMetadata: reverted (0.0052s) ==================

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Darby Frey

Merge request reports