Restore backup is broken

I think this commit broke restoring via UI. I get the error below.

Traceback (most recent call last):
	5: from /src/script/spawn_backup_restore.rb:4:in `<main>'
	4: from /src/script/spawn_backup_restore.rb:4:in `fork'
	3: from /src/script/spawn_backup_restore.rb:36:in `block in <main>'
	2: from /src/script/spawn_backup_restore.rb:15:in `restore'
	1: from /src/script/spawn_backup_restore.rb:15:in `new'
/src/lib/backup_restore/restorer.rb:12:in `initialize': missing keyword: :location (ArgumentError)

It looks like spawn_backup_restore.rb just needs a location:nil added to the constructor maybe?

3 Likes

I’m sorry about that. I’ll fix it on Monday. You can work around it by using the CLI for now: Restore a backup from command line

8 Likes

Fixed in https://github.com/discourse/discourse/commit/4b05fc2d2dea2d1ceefbc470730d950439959fff

7 Likes