[Bug]: Import hangs #1449

Closed
opened 2026-02-28 19:43:52 -06:00 by GiteaMirror · 8 comments
Owner

Originally created by @fantasygoat on GitHub (Sep 27, 2024).

Verified issue does not already exist?

  • I have searched and found no existing issue
  • I will be providing steps how to reproduce the bug (in most cases this will also mean uploading a demo budget file)

What happened?

Fresh docker-based server setup. Imported YNAB4 budget without issue. Attempting to import a file, clicking the Import link and selecting a quicken file leads to a black box with spinner that never finishes.

Where are you hosting Actual?

Docker

What browsers are you seeing the problem on?

Chrome, Safari

Operating System

Mac OSX

Originally created by @fantasygoat on GitHub (Sep 27, 2024). ### Verified issue does not already exist? - [X] I have searched and found no existing issue - [X] I will be providing steps how to reproduce the bug (in most cases this will also mean uploading a demo budget file) ### What happened? Fresh docker-based server setup. Imported YNAB4 budget without issue. Attempting to import a file, clicking the Import link and selecting a quicken file leads to a black box with spinner that never finishes. ### Where are you hosting Actual? Docker ### What browsers are you seeing the problem on? Chrome, Safari ### Operating System Mac OSX
GiteaMirror added the bug label 2026-02-28 19:43:52 -06:00
Author
Owner

@fantasygoat commented on GitHub (Sep 27, 2024):

I just did a CSV import and it worked. Seems to not like Quicken files.

@fantasygoat commented on GitHub (Sep 27, 2024): I just did a CSV import and it worked. Seems to not like Quicken files.
Author
Owner

@MatissJanis commented on GitHub (Sep 28, 2024):

👋 Would you mind providing an example import file so we could try and reproduce?

@MatissJanis commented on GitHub (Sep 28, 2024): 👋 Would you mind providing an example import file so we could try and reproduce?
Author
Owner

@fantasygoat commented on GitHub (Sep 28, 2024):

accountactivity-7.qfx.zip

@fantasygoat commented on GitHub (Sep 28, 2024): [accountactivity-7.qfx.zip](https://github.com/user-attachments/files/17176769/accountactivity-7.qfx.zip)
Author
Owner

@MatissJanis commented on GitHub (Oct 2, 2024):

Thanks! I was able to reproduce and patch the issue here: https://github.com/actualbudget/actual/pull/3541

However, sadly your file is malformed so it will still not work when importing in Actual. But at least now it will show an error message.

@MatissJanis commented on GitHub (Oct 2, 2024): Thanks! I was able to reproduce and patch the issue here: https://github.com/actualbudget/actual/pull/3541 However, sadly your file is malformed so it will still not work when importing in Actual. But at least now it will show an error message.
Author
Owner

@fantasygoat commented on GitHub (Oct 2, 2024):

How is it malformed? It's from my bank and it loads in NYAB4 and NYAB Web just fine. So basically I can't use my bank's downloaded files?

@fantasygoat commented on GitHub (Oct 2, 2024): How is it malformed? It's from my bank and it loads in NYAB4 and NYAB Web just fine. So basically I can't use my bank's downloaded files?
Author
Owner

@MatissJanis commented on GitHub (Oct 2, 2024):

The problem is on line 38: <ACCTTYPE>. Remove that empty tag and your imports will work 👍

@MatissJanis commented on GitHub (Oct 2, 2024): The problem is on line 38: `<ACCTTYPE>`. Remove that empty tag and your imports will work :+1:
Author
Owner

@fantasygoat commented on GitHub (Oct 2, 2024):

Wouldn't it be better to ignore empty tags? I can pre-process the files I guess but that seems needlessly painful.

@fantasygoat commented on GitHub (Oct 2, 2024): Wouldn't it be better to ignore empty tags? I can pre-process the files I guess but that seems needlessly painful.
Author
Owner

@MatissJanis commented on GitHub (Oct 2, 2024):

Technically <ACCTYPE> is invalid XML syntax for empty values. The correct syntax would be either </ACCTYPE> or <ACCTYPE></ACCTYPE>. So the XML parser correctly breaks when it parses the file.

Though I can see that it would be better in this case to auto-magically ignore such elements. Feel free to either contribute with a PR or submit a feature request for this.

@MatissJanis commented on GitHub (Oct 2, 2024): Technically `<ACCTYPE>` is invalid XML syntax for empty values. The correct syntax would be either `</ACCTYPE>` or `<ACCTYPE></ACCTYPE>`. So the XML parser correctly _breaks_ when it parses the file. Though I can see that it would be better in this case to auto-magically ignore such elements. Feel free to either contribute with a PR or submit a feature request for this.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#1449