Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KeePass XC Import fails on KeePassXC data export; Generic CSV Imports Partial Data #762

Open
tekreativ opened this issue Mar 18, 2023 · 1 comment
Labels

Comments

@tekreativ
Copy link

tekreativ commented Mar 18, 2023

Bug report

Steps to reproduce

  1. In KeePassXC,
    a. Create a password vault with this structure (TestwordsExport2.csv):
    - A Root folder containing Account 1, Account 2 and Group 1
    - To Group 1, add Account 3 and Group 2
    - To Group 2, add Account 4

    b. Export CSV (no options presented)

  2. In Passman
    a. Create a vault "VaultName".
    b. Go to Settings> Settings> Import credentials> Import type = 'KeePass csv' and select Import.
    c. Select Choose File and select the exported CSV, and click the Import button.
    d. Select VaultName from the breadcrumb menu and the Refresh button to view the imported credentials.
    Note: Passman lists no credentials in the vault

    e. Go to Settings> Settings> Import credentials> and select the link for the generic CSV importer.
    f. Choose the Select File button, the KeePass CSV file, and the Start import button
    Note: Passman displays the progress of the import

    g. Select VaultName from the breadcrumb menu and the Refresh button to view the imported credentials.
    Note: Passman lists 3 credentials in the vault.

  3. Repeat the above

Expected behaviour

  • In Step 3b, above, the Read progress should be "3/3"
  • In Step 3c, above, Passman should display 3 passwords and 2 groups
  • In Step 3f, above, Passman should display the progress of the import

Actual behaviour

  • In Step 3b, above, Passman reports, "Read progress 3/4" and "File read"
  • In Step 3c, above, Passman displays, "It does not seem that you have any passwords. Do you want to add one?"
  • In Step 3f, above, Passman displays the progress as expected:
Adding Testword 1
Added Testword 1
Adding Testword 2
Added Testword 2
Adding Testword 3
Added Testword 3
Done

Configuration

Operating system:

  • Elementary OS 6.1
  • Ubuntu 20.04.5 LTS
  • Linux 5.15.0-67 generic kernel

Browser: Ungoogled Chromium 110.0.5481.177 (Official Build, ungoogled-chromium) (64-bit)

Extensions that might cause interference: None installed

Passman version: An instance of your demo server at https://demo.passman.cc

  • GitHub version: 2.4.3 ↗
  • Local version: 2.4.3

Operating system: Unknown

Web server: Unknown

Database: Unknown

PHP version: Unknown

cloud server: Nextcloud

cloud version: 21.0.9

Browser log

I don't believe Ungoogled Chromium keeps logs. At least, I can't find them. ``` Insert your browser log here, this could for example include:

a) The javascript console log for

No OC found
Fr @ index.js:46
jquery-migrate.min.js:2 JQMIGRATE: Migrate is installed, version 3.3.2
globals.js:60 jQuery is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
globals.js:60 $ is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
globals.js:60 jQuery is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
globals.js:60 Clipboard is deprecated: please ship your own, this will be removed in Nextcloud 20
pe @ globals.js:60
4globals.js:60 jQuery is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
crypto_wrap.js?v=534ac216-0:146 Crypto stuff initialized
initEngines @ crypto_wrap.js?v=534ac216-0:146
globals.js:60 jQuery is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
13globals.js:60 $ is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
session-heartbeat.js:97 session heartbeat polling started
2globals.js:60 $ is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
globals.js:60 oc_requesttoken is deprecated: use OC.requestToken instead, this will be removed in Nextcloud 20
pe @ globals.js:60
2globals.js:60 $ is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
2globals.js:60 jQuery is deprecated: The global jQuery is deprecated. It will be updated to v3.x in Nextcloud 21. In later versions of Nextcloud it might be removed completely. Please ship your own.
pe @ globals.js:60
index.js:311 OC.requestToken changed TIp85PeKTOfFK/rSD/hohimB4LbtDTEQjEfkN+FlGiM=:GN85vZTPI6uWZ4K/e6kJw3HAjN/aPHxx/j++B6wyakc=

b) The network log
c) ...

</details>   


#### Cloud log
<!--
The log has the name owncloud.log or nextcloud.log and is found in the data folder
-->

<details>
<summary>I don't believe I can access the data folder on your demo server.</summary>

Insert your cloud log here

</details>


## Feature request

**User type**: Logged-in or public user

**User level**: Beginners, intermediate or advanced

### Description
<!--
Please try to give as much information as you can about your request
-->

### Benefit / value
<!--
Please explain how it could benefit users of the app, other apps or 3rd party services
-->

### Risk / caveats
<!--
Please explain the risks and caveats associated with this request
-->

### Sponsorship
<!--
This greatly accelerates the delivery of a feature
-->
 
**Are you a developer willing to implement this feature?**: yes/no

**Can you sponsor the development of this feature or do you know someone who can?**: yes/no
@binsky08 binsky08 added the bug label May 3, 2023
@djibux
Copy link

djibux commented Jun 14, 2023

I am experiencing that issue as well. Generic CSV import works but with some side effects: OTP are broken for example, but can be fixed by adjusting secret.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants