Skip to content

feat(pam): individual resource & account pages#5376

Merged
x032205 merged 10 commits intomainfrom
pam/windows/single-page-view
Feb 5, 2026
Merged

feat(pam): individual resource & account pages#5376
x032205 merged 10 commits intomainfrom
pam/windows/single-page-view

Conversation

@x032205
Copy link
Member

@x032205 x032205 commented Feb 5, 2026

Context

This PR adds individual resource and account pages. Clicking on a resource from the resources table takes you into the individual resources page which lists the accounts under that resource.

Clicking on an account from the individual resources page takes you to the individual account page, which lists resources that the account can connect to (just the parent for now).

Accessing the individual account page from the actual accounts tab in the PAM project is not supported due to upcoming resource-first restructuring of PAM.

Screenshots

CleanShot 2026-02-04 at 22 16 47@2x CleanShot 2026-02-04 at 22 17 06@2x

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • Docs
  • Chore

Checklist

  • Title follows the conventional commit format: type(scope): short description (scope is optional, e.g., fix: prevent crash on sync or fix(api): handle null response).
  • Tested locally
  • Updated docs (if needed)
  • Read the contributing guide

@maidul98
Copy link
Collaborator

maidul98 commented Feb 5, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 5, 2026

Greptile Overview

Greptile Summary

This PR adds individual resource and account detail pages to the PAM (Privileged Access Management) system, enabling users to navigate from the resources table to view detailed information about specific resources and their associated accounts.

Key Changes:

  • Added new GET /api/v1/pam/accounts/:accountId endpoint with proper authentication, permission checks, and audit logging
  • Created getById service method in pam-account-service.ts with resource details retrieval and decryption
  • Added findByIdWithResourceDetails DAL method with proper table joins using qualified column names
  • Implemented new frontend pages: PamResourceByIDPage and PamAccountByIDPage with tabbed interfaces
  • Added navigation flow: Resources table → Resource detail page → Account detail page
  • Proper permission checks applied throughout using ProjectPermissionCan components
  • All changes maintain existing security patterns and audit logging requirements

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • All changes follow established patterns for authentication, authorization, and audit logging. The new API endpoint properly validates inputs, checks permissions, and logs access. Database queries use proper table-qualified column names in joins. Frontend components implement appropriate permission checks and error handling.
  • No files require special attention

Important Files Changed

Filename Overview
backend/src/ee/routes/v1/pam-account-routers/pam-account-router.ts Added new GET /:accountId endpoint with proper authentication, audit logging, and permission checks
backend/src/ee/services/pam-account/pam-account-service.ts Added getById method with proper permission checks and account decryption logic
backend/src/ee/services/pam-account/pam-account-dal.ts Added findByIdWithResourceDetails method with proper JOIN and column references
frontend/src/pages/pam/PamAccountByIDPage/PamAccountByIDPage.tsx New page component displaying individual account details with proper permission checks and access controls
frontend/src/pages/pam/PamResourceByIDPage/PamResourceByIDPage.tsx New page component displaying individual resource details with accounts table
frontend/src/pages/pam/PamResourceByIDPage/components/PamResourceAccountsSection.tsx Accounts section component with navigation to individual account pages

@x032205 x032205 merged commit 82903f6 into main Feb 5, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants