vaultwarden.models.bitwarden.get_organization() requires owner user

Open
#39 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
45/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
python
Domain
api, backend

Research direction

Start at vaultwarden.models.bitwarden.get_organization() and inspect how bitwarden_client data from the /api/sync endpoint is represented, along with the Organization object shown in the issue. Verify the behavior with a normal user and confirm that organization access no longer requires the owner-only endpoint.

Written by the indexing model from the issue text.

Description

enhancement good first issue help wanted

trying to access my organization from a normal user fails:

from vaultwarden.models.bitwarden import get_organization
orga= get_organization(bitwarden_client, org_uuid)
GET /api/organizations/aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee
Unauthorized Error: You need to be Owner to call this endpoint
Request guard `OwnerHeaders` failed: "You need to be Owner to call this endpoint".
(get_organization) GET /api/organizations/<org_id> => 401 Unauthorized

it seems that it is not necessary to call this endpoint. bitwarden_client should already have all the details about a user's organization (from the /api/sync endpoint). alternatively, an Organization object could be instantiated manually, but that seems more like a workaround than a solution:

org = Organization(
    Id = "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee",
    Name = "my Org",
    Object = None,
    bitwarden_client = bitwarden_client
)
Dominant language
Python
Stars
37
Forks
13
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from numberly/python-vaultwarden

All issues in numberly/python-vaultwarden

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.