Project

General

Profile

Actions
Copy link

Task #3055

closed

Allow the execution of only selected reports for users

Added by Tomáš Doischer almost 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
High
Assignee:
Tomáš Doischer
Category:
Report
Target version:
12.2.0
Start date:
02/11/2022
Due date:
% Done:

100%

Estimated time:
Owner:

Description

It is often not desirable to allow each user to see and run the set of reports. Prepare an evaluator to allow using only permitted reports.

Actions
Copy link
 #1

Updated by Tomáš Doischer almost 3 years ago

Needed for #3054.

Actions
Copy link
 #2

Updated by Tomáš Doischer almost 3 years ago

  • Subject changed from Create codeable report evaluator to Allow the execution of only selected reports for users
Actions
Copy link
 #3

Updated by Tomáš Doischer almost 3 years ago

  • Sprint set to Sprint 1 (úno 18 - bře 02)
Actions
Copy link
 #4

Updated by Tomáš Doischer almost 3 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #5

Updated by Tomáš Doischer almost 3 years ago

  • Status changed from In Progress to Needs feedback
  • % Done changed from 0 to 80

Implemented. The issue is that reports are just beans and we have no way of adding permissions for them at the moment. To solve the problem, we limit which reports are returned as available.

PR: https://github.com/bcvsolutions/CzechIdMng/pull/179

Documentation: https://wiki.czechidm.com/devel/documentation/security/dev/authorization#reportbyreporttypeevaluator

Actions
Copy link
 #6

Updated by Tomáš Doischer almost 3 years ago

  • Assignee changed from Tomáš Doischer to Roman Kučera

@kucerar, can you please give me feedback?

Actions
Copy link
 #7

Updated by Roman Kučera almost 3 years ago

  • Sprint changed from Sprint 12.2-1 (Feb 18 - Mar 02) to Sprint 12.2-2 (Mar 02 - Mar 16)
Actions
Copy link
 #8

Updated by Roman Kučera almost 3 years ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Roman Kučera to Tomáš Doischer

@doischert I added some comments to pull request. Please look at it. It's no big deal, only minor things

Actions
Copy link
 #9

Updated by Tomáš Doischer over 2 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Tomáš Doischer to Roman Kučera
  • % Done changed from 80 to 100

Thanks for your feedback, I made the suggested changes. Can you look at it again?

Actions
Copy link
 #10

Updated by Roman Kučera over 2 years ago

  • Assignee changed from Roman Kučera to Tomáš Doischer

One last typo then you can merge it. Thx

Actions
Copy link
 #11

Updated by Tomáš Doischer over 2 years ago

  • Status changed from Needs feedback to Resolved

OMG, thanks, fixed and merged to develop.

Actions
Copy link
 #12

Updated by Roman Kučera over 2 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF