Skip to content

Commit

Permalink
Increase database field size for notificationaddress, to allow storin…
Browse files Browse the repository at this point in the history
…g actual FCM addresses, instead of (shorter) TX addresses

Co-authored-by: Thijs Kinkhorst <[email protected]>
  • Loading branch information
phavekes and thijskh committed Feb 16, 2024
1 parent 7de3265 commit 67732ec
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion config/db/mysql-create-tables.sql
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ CREATE TABLE IF NOT EXISTS user (
tmpblockattempts integer,
blocked tinyint(1),
notificationtype varchar(10),
notificationaddress varchar(64)
notificationaddress varchar(256)
);

CREATE TABLE IF NOT EXISTS tiqrstate (
Expand Down
5 changes: 5 additions & 0 deletions config/db/mysql-upgrade-notificationaddress.sql
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/*
Initially the notificationaddress column was created as varchar(64). That is fine for
tokenaxchange addresses, but too small for FCM addresses.
*/
ALTER TABLE user MODIFY notificationaddress VARCHAR (256);

0 comments on commit 67732ec

Please sign in to comment.