Skip to content

Feat/devsu 2815 make create report access project specific #94

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.

Sign up for GitHub

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

Open
sshugsc wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Feat/devsu 2815 make create report access project specific #94

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a1968ed
checking for project membership before creating the report
sshugsc Jan 29, 2026
be3add7
lint
sshugsc Jan 30, 2026
ce7460a
Merge branch 'develop' into feat/DEVSU-2815-make-create-report-access…
sshugsc Feb 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions pori_python/ipr/connection.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,10 @@ def upload_report(

projects = self.get("project")
project_names = [item["name"] for item in projects]
project_users = {
item["name"]: [user["username"] for user in item.get("users", [])]
for item in projects
}

# if project is not exist, create one
if content["project"] not in project_names:
Expand All @@ -118,6 +122,11 @@ def upload_report(
except Exception as err:
raise Exception(f"Project creation failed due to {err}")

if self.username not in project_users[content["project"]]:
raise Exception(
f"User have no permission to create report in project {content['project']}"
)

if ignore_extra_fields:
initial_result = self.post("reports-async?ignore_extra_fields=true", content)
else:
Expand Down