Skip to content

scripts for key and credential migration - #192

Merged
briskt merged 2 commits into
mainfrom
data-migration
Aug 5, 2026
Merged

scripts for key and credential migration#192
briskt merged 2 commits into
mainfrom
data-migration

Conversation

@briskt

@briskt briskt commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

IDP-2161 Plan moving Wycliffe's 2SV data


Added

  • Added data migration scripts to copy API keys and TOTP/WebAuthn credentials from one set of tables to another.

@briskt
briskt requested a review from a team as a code owner August 3, 2026 09:41
@briskt
briskt requested review from ethancanne, hobbitronics, jason-jackson and samuel-sowpati-sil and removed request for a team August 3, 2026 09:41
@briskt

briskt commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

I used a combination of Gemini, Copilot, and Claude to generate this code. I may have even contributed a few lines myself. I tested it by moving data from the dev (staging) tables into tables running locally on my machine. For final testing, I used the SIL credentials from the production tables and uploaded to the same local DynamoDB container I used for dev credentials. There were no performance or memory issues, but it did seem to slow down slightly as it neared completion of the upload of the production data set.

@hobbitronics hobbitronics left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a hard time reading bash of this complexity and am not familiar with much of what's going on here, but the scripts share the same shape which helps and tests seem valid. I asked about the commands and Copliot had some concerns which may or may not be valid.

Comment thread scripts/fetch_key.sh Outdated
Comment thread scripts/upload_creds.sh
@briskt
briskt merged commit a5daf4c into main Aug 5, 2026
5 checks passed
@briskt
briskt deleted the data-migration branch August 5, 2026 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants