Project

General

Profile

Feature #27450

Check a user's permission before allowing them to access react pages

Added by Walden Raines over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web UI
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Currently we allow users to visit pages they don't have access to. If a user doesn't have permission to view a page we shouldn't allow them to view it.

This should be done for all react pages in katello.


Related issues

Related to Katello - Bug #25113: Subscriptions details page does not handle Org switch correctly.Closed
Copied to Foreman - Feature #27644: Check a user's permission before allowing them to access react pages Closed

Associated revisions

Revision 97de7bdb (diff)
Added by Jeremy Lenz about 1 year ago

Fixes #27450: Add PermissionDenied component (#8262)

clean up component
add PermissionDenied to Subs page
Refactor RedHatRepositories Reducer
fix tests & snapshots
code style fixes
use block instead of disabling Eslint rule
implement code style suggestions
Remove component from katello and instead import from foremanReact
Address 'Flow 1' with error messaging & other improvements
update tests & snapshots
update Jest mock
refactor RedHatRepositoriesPage folder structure
re-fix PropTypes recommended issue
add RedHatRepositoriesPage test to increase coverage
add second block instead of eslint-disable
(some) changes after laviro review
rebase

History

#1 Updated by Walden Raines over 1 year ago

  • Related to Bug #25113: Subscriptions details page does not handle Org switch correctly. added

#2 Updated by Justin Sherrill over 1 year ago

  • Triaged changed from No to Yes
  • Target version set to Katello 3.14.0

#3 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/8262 added

#4 Updated by Jeremy Lenz over 1 year ago

  • Copied to Feature #27644: Check a user's permission before allowing them to access react pages added

#5 Updated by The Foreman Bot about 1 year ago

  • Fixed in Releases Katello 3.14.0 added

#6 Updated by Jeremy Lenz about 1 year ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF