Merge branch 'master' into fix/update-requires-python-minimum

Author: mr-raj12

.github/actions/setup/action.yml

@@ -43,7 +43,7 @@ runs:
 
     - name: Caching for Pre-Commit
       if: ${{ inputs.pre-commit }}
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       with:
         path: ~/.cache/pre-commit
         key: pre-commit|${{ env.PY }}|${{ hashFiles('.pre-commit-config.yaml') }}

.github/workflows/build-macos.yml

@@ -26,7 +26,7 @@ jobs:
 
     steps:
     - name: Check out selected branch
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
       with:
         ref: ${{ github.event.inputs.branch }}
     - name: Install uv

.github/workflows/test.yml

@@ -20,7 +20,7 @@ jobs:
   lint:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Setup python, vorta and dev deps
         uses: ./.github/actions/setup
         with:
@@ -37,7 +37,7 @@ jobs:
       matrix-unit: ${{ steps.set-matrix-unit.outputs.matrix }}
       matrix-integration: ${{ steps.set-matrix-integration.outputs.matrix }}
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
       - name: Give execute permission to script
         run: chmod +x ./.github/scripts/generate-matrix.sh
@@ -63,7 +63,7 @@ jobs:
       matrix: ${{fromJson(needs.prepare-matrix.outputs.matrix-unit)}}
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
       - name: Install system dependencies
         uses: ./.github/actions/install-dependencies
@@ -99,7 +99,7 @@ jobs:
         run: echo $PKG_CONFIG_PATH && make test-unit
 
       - name: Upload coverage to Codecov
-        uses: codecov/codecov-action@v3
+        uses: codecov/codecov-action@v4
         env:
           OS: ${{ runner.os }}
           python: ${{ matrix.python-version }}
@@ -116,7 +116,7 @@ jobs:
       matrix: ${{fromJson(needs.prepare-matrix.outputs.matrix-integration)}}
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
       - name: Install system dependencies
         uses: ./.github/actions/install-dependencies

src/vorta/keyring/darwin.py

@@ -94,9 +94,9 @@ def delete_password(self, service, repo_url):
         try:
             result, password_length, password_buffer, keychain_item = SecKeychainFindGenericPassword(
                 self.login_keychain,
-                len(service),
+                len(service.encode()),
                 service.encode(),
-                len(repo_url),
+                len(repo_url.encode()),
                 repo_url.encode(),
                 None,
                 None,
@@ -124,9 +124,9 @@ def get_password(self, service, repo_url):
             keychain_item,
         ) = SecKeychainFindGenericPassword(
             self.login_keychain,
-            len(service),
+            len(service.encode()),
             service.encode(),
-            len(repo_url),
+            len(repo_url.encode()),
             repo_url.encode(),
             None,
             None,

tests/unit/test_keyring_darwin.py

@@ -0,0 +1,16 @@
+import sys
+import uuid
+
+import pytest
+
+
+@pytest.mark.skipif(sys.platform != "darwin", reason="macOS only")
+def test_keyring_non_ascii_repo():
+    from vorta.keyring.darwin import VortaDarwinKeyring
+
+    UNICODE_PW = 'password'
+    REPO = 'vorta-test-repo-한글'
+
+    keyring = VortaDarwinKeyring()
+    keyring.set_password('vorta-repo', REPO, UNICODE_PW)
+    assert keyring.get_password("vorta-repo", REPO) == UNICODE_PW