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 19, 2024
1 parent a497f26 commit 26962da
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 26962da

Please sign in to comment.